[PATCH] D36451: [ELF, LinkerScript] Support ! operator in linker script.

Rui Ueyama via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 9 11:47:19 PDT 2017


ruiu added inline comments.


================
Comment at: ELF/ScriptLexer.cpp:215-217
+  // Don't breakup != operator into 2 tokens
+  if (V.size() == 2 && V[0] == "!" && V[1] == "=")
+    return;
----------------
I don't think this is correct. For example, I don't think this can handle expression `0!=1`. You want to add code to tokenizeExpr instead of this function.


https://reviews.llvm.org/D36451





More information about the llvm-commits mailing list