[PATCH] D48153: Add TARGET(foo) linker script directive.

George Rimar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 18 06:37:52 PDT 2018


grimar accepted this revision.
grimar added a comment.

This is probably OK. LGTM with a nit.



================
Comment at: lld/ELF/ScriptParser.cpp:491
+  else
+    setError("unknown target: " + Tok);
+}
----------------
Can you add a test for that error?


https://reviews.llvm.org/D48153





More information about the llvm-commits mailing list