[llvm-commits] [PATCH] [lld] Changes for adding hexagon target and add noinhibit-exec option

[Sean Silva]

  Is this flag platform-agnostic? Does it make sense for there to be code in LinkerOptions.h to support this flag?

  I just want to avoid a rat's nest down the road where every feature of very linker adds a new member to LinkerOptions.


================
Comment at: test/Driver/hexagon-target.test:1-2
@@ +1,3 @@
+# Will fail with unresolved symbol 
+RUN: lld -flavor ld -target hexagon || exit 0 
+# Will say unresolved symbols but not fail
----------------
This command will never fail. (try running `(/bin/false || exit 0)` and `(/bin/true || exit 0)`). How is it supposed to test anything?


http://llvm-reviews.chandlerc.com/D282