« first day (23 days earlier)      last day (17 days later) » 

1:05 PM
Pushed:some more.
[master] Make Simplify transform in-place
 * [move-semantics] Move semantics in AstGen (ew)
 * [move-semantics^] Move semantics in semantic actions
TL;DR the move-semantics didn't show a big improvement. The raised complexity for AstGen's move semantics was substantial. So I'd suggest to not merge the move-semantics branch for simplicity.
The interesting thing is that move-semantics were already being invoked in the non-explicitly move-enabled version of the code (to an extent). This is likely the result of good compiler optimization and pass-by-value style
Oh, and that "<>" backtracking issue has gone away with 1fe8931 Flatten the term_ rule. Because now-adays all binary operators (except assignment) are recognized without back-tracking and the make_<Binary> composes the AST with proper precedence instead of differentiating in the rule.
@llm I think we're done here? I can't immediately think of essential ground to cover (without getting into your specific application domain, perhaps)
 
 
4 hours later…
5:19 PM
To quantify that de flattened term_rul indeed removed inefficiencies, I added some statistics github.com/sehe/qi-extended-parser-evaluator/commit/…
 
 
2 hours later…
6:55 PM
This gives me one idea (prevent rule invocations that do not succeed, so look-ahead for unary_ and quoted_string)
 
 
2 hours later…
8:28 PM
Achieved, but not without some tweaks around the edges.
The tweak has to do with sub-Expressions suddenly being removed (so the precedences would be broken in some cases). The low-tech solution to this has been to replace recursive_variant_ (coding for recursive_wrapper<Ast::Expression>) with a separate type Ast::SubExpression which doesn't have the copy-construction issue
 

« first day (23 days earlier)      last day (17 days later) »