[PATCH] D36508: [ELF] - Linkerscript: Add `~` as separate math token.

George Rimar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 11 01:25:01 PDT 2017


grimar added a comment.

- Rebased.



================
Comment at: ELF/ScriptLexer.cpp:171
 static std::vector<StringRef> tokenizeExpr(StringRef S) {
-  StringRef Ops = "+-*/:"; // List of operators
+  StringRef Ops = "+-*/:~"; // List of operators
 
----------------
ruiu wrote:
> I'd think you should spend a little more time on each patch. Apparently, you need to update the following comment that assumes `+-*/` is a list of operators.
It was already out of sync, fixed in rL315442



https://reviews.llvm.org/D36508





More information about the llvm-commits mailing list