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

Hafiz Abid Qadeer via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 10 02:02:16 PDT 2017


abidh 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;
----------------
ruiu wrote:
> 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.
I think 0!=1 was failing even without my patch. I have fixed it now and added a test to check it.


https://reviews.llvm.org/D36451





More information about the llvm-commits mailing list