aboutsummaryrefslogtreecommitdiff
path: root/libctf
diff options
context:
space:
mode:
authorJan Beulich <jbeulich@suse.com>2022-01-04 10:05:17 +0100
committerJan Beulich <jbeulich@suse.com>2022-01-04 10:05:17 +0100
commit5ed4d49d107c46670a7994711f0284776a35284e (patch)
treed883166be8b96d7aa83f6c1f7dfc0d1ebfbc6c53 /libctf
parent79541a6d9220e800ebc5278594105982d6e1d80c (diff)
downloadbinutils-5ed4d49d107c46670a7994711f0284776a35284e.zip
binutils-5ed4d49d107c46670a7994711f0284776a35284e.tar.gz
binutils-5ed4d49d107c46670a7994711f0284776a35284e.tar.bz2
gas: rework handling of backslashes in quoted symbol names
Strange effects can result from the present handling, e.g.: .if 1 "backslash\\": .endif yields first (correctly) "missing closing `"'" but then also "invalid character '\' in mnemonic" and further "end of file inside conditional". Symbols names ending in \ are in principle not expressable with that scheme. Instead of recording whether a backslash was seen, inspect the subsequent character right away. Only accept \\ (meaning a single backslash in the resulting symbol name) and \" (meaning an embedded double quote in the resulting symbol name) for now, warning about any other combination. While perhaps not necessary immediately, also permit concatenated strings to form a symbol name. This may become useful if going forward we would want to support \<octal> or \x<hex> sequences, where closing and re-opening quotes can be useful to delimit such sequences. The ELF "Multibyte symbol names" test gets switched away from using .set, as that would now also mean excluding nios2 and pru. By using .equiv instead, even the existing #notarget can be dropped. (For h8300 the .section directive additionally needs attributes specified, to avoid a target specific warning.)
Diffstat (limited to 'libctf')
0 files changed, 0 insertions, 0 deletions