aboutsummaryrefslogtreecommitdiff
path: root/ld/ldlex.h
diff options
context:
space:
mode:
authorAlan Modra <amodra@gmail.com>2021-02-01 09:15:41 +1030
committerAlan Modra <amodra@gmail.com>2021-02-02 01:27:12 +1030
commit40726f16a8d7105761e36398054860a923d4efc9 (patch)
tree06d77510d97baeed95b4e7f56c0c20ed5cc8bdad /ld/ldlex.h
parent82a1fd3a4935fe665cf08bc6820942c4a091184c (diff)
downloadgdb-40726f16a8d7105761e36398054860a923d4efc9.zip
gdb-40726f16a8d7105761e36398054860a923d4efc9.tar.gz
gdb-40726f16a8d7105761e36398054860a923d4efc9.tar.bz2
ld script expression parsing
Parsing symbol or file/section names in ld linker scripts is a little complicated. Inside SECTIONS, a name might be the start of an expression or an output section. Is ".foo=x-y" a fancy section name or is it the expression ".foo = x - y"? It isn't possible for a single lookahead parser to decide, so the answer in this case is that it's a section name. This is the reason why everyone writes linker script assignment expressions with lots of white-space. However, there are many places where the parser knows for sure that an expression is expected. Those could be written without whitespace given the first change to ldlex.l below. Unfortunately, that runs into a lookahead problem. Optional expressions at the end of an output section statement require the parser to look ahead one token in expression context. For this example from standard scripts .interp : { *(.interp) } .note.gnu.build-id : { *(.note.gnu.build-id) } at the end of the .interp closing brace, the parser is looking for a possible memspec, phdr, fill or even an optional comma. The next token is a NAME, but in expression context that NAME now doesn't include '-' as a valid char. So the lookahead NAME is ".note.gnu.build" with an unexpected "-id" syntax error before the colon. The rest of the patch involving ldlex_backup arranges to discard that NAME token so that it will be rescanned in the proper script context. * ldgram.y (section): Call ldlex_backup. Remove empty action. * ldlex.h (ldlex_backup): Declare. * ldlex.l (<EXPRESSION>NAME): Don't use NOCFILENAMECHAR set of chars, use SYMBOLNAMECHAR. (ldlex_backup): New function.
Diffstat (limited to 'ld/ldlex.h')
-rw-r--r--ld/ldlex.h1
1 files changed, 1 insertions, 0 deletions
diff --git a/ld/ldlex.h b/ld/ldlex.h
index b010102..d9b36ea 100644
--- a/ld/ldlex.h
+++ b/ld/ldlex.h
@@ -191,6 +191,7 @@ extern void ldlex_defsym (void);
extern void ldlex_expression (void);
extern void ldlex_both (void);
extern void ldlex_popstate (void);
+extern void ldlex_backup (void);
extern const char* ldlex_filename (void);
/* In lexsup.c. */