diff options
author | Eelco Dolstra <eelco.dolstra@logicblox.com> | 2014-10-04T23·04+0200 |
---|---|---|
committer | Eelco Dolstra <eelco.dolstra@logicblox.com> | 2014-10-04T23·04+0200 |
commit | b6809608cc467925db44b1eb435095c37e433255 (patch) | |
tree | 65eb05089cc9c291febd85a25e917fb12f52e8ff /src/libexpr/parser.y | |
parent | 14188069692215264eb1a25d60920fac7117a2ef (diff) |
Get rid of some unnecessary ExprConcatStrings nodes in dynamic attrs
This gives a ~18% speedup in NixOS evaluation (after converting most calls to hasAttr/getAttr to dynamic attrs).
Diffstat (limited to '')
-rw-r--r-- | src/libexpr/parser.y | 20 |
1 files changed, 8 insertions, 12 deletions
diff --git a/src/libexpr/parser.y b/src/libexpr/parser.y index 60a87ca94e10..dcb270b862a3 100644 --- a/src/libexpr/parser.y +++ b/src/libexpr/parser.y @@ -341,11 +341,7 @@ expr_op | expr_op UPDATE expr_op { $$ = new ExprOpUpdate(CUR_POS, $1, $3); } | expr_op '?' attrpath { $$ = new ExprOpHasAttr($1, *$3); } | expr_op '+' expr_op - { vector<Expr *> * l = new vector<Expr *>; - l->push_back($1); - l->push_back($3); - $$ = new ExprConcatStrings(CUR_POS, false, l); - } + { $$ = new ExprConcatStrings(CUR_POS, false, new vector<Expr *>({$1, $3})); } | expr_op '-' expr_op { $$ = new ExprApp(CUR_POS, new ExprApp(new ExprVar(data->symbols.create("__sub")), $1), $3); } | expr_op '*' expr_op { $$ = new ExprApp(CUR_POS, new ExprApp(new ExprVar(data->symbols.create("__mul")), $1), $3); } | expr_op '/' expr_op { $$ = new ExprApp(CUR_POS, new ExprApp(new ExprVar(data->symbols.create("__div")), $1), $3); } @@ -461,13 +457,13 @@ attrs : attrs attr { $$ = $1; $1->push_back(AttrName(data->symbols.create($2))); } | attrs string_attr { $$ = $1; - ExprString *str = dynamic_cast<ExprString *>($2); + ExprString * str = dynamic_cast<ExprString *>($2); if (str) { $$->push_back(AttrName(str->s)); delete str; } else - throw ParseError(format("dynamic attributes not allowed in inherit at %1%") - % makeCurPos(@2, data)); + throw ParseError(format("dynamic attributes not allowed in inherit at %1%") + % makeCurPos(@2, data)); } | { $$ = new AttrPath; } ; @@ -476,12 +472,12 @@ attrpath : attrpath '.' attr { $$ = $1; $1->push_back(AttrName(data->symbols.create($3))); } | attrpath '.' string_attr { $$ = $1; - ExprString *str = dynamic_cast<ExprString *>($3); + ExprString * str = dynamic_cast<ExprString *>($3); if (str) { $$->push_back(AttrName(str->s)); delete str; } else - $$->push_back(AttrName(static_cast<ExprConcatStrings *>($3))); + $$->push_back(AttrName($3)); } | attr { $$ = new vector<AttrName>; $$->push_back(AttrName(data->symbols.create($1))); } | string_attr @@ -491,7 +487,7 @@ attrpath $$->push_back(AttrName(str->s)); delete str; } else - $$->push_back(AttrName(static_cast<ExprConcatStrings *>($1))); + $$->push_back(AttrName($1)); } ; @@ -502,7 +498,7 @@ attr string_attr : '"' string_parts '"' { $$ = $2; } - | DOLLAR_CURLY expr '}' { $$ = new ExprConcatStrings(CUR_POS, true, new vector<Expr*>(1, $2)); } + | DOLLAR_CURLY expr '}' { $$ = $2; } ; expr_list |