eugenis added a comment.

The driver seems to always add sanitizer runtimes first on the linker command 
line.



================
Comment at: clang/lib/Driver/ToolChains/CommonArgs.cpp:842
+      if (!Args.hasArg(options::OPT_shared))
+        SharedRuntimes.push_back("hwasan-preinit");
     }
----------------
pcc wrote:
> Shouldn't it be added to `HelperStaticRuntimes`? Otherwise I'm not sure how 
> this would work because the library would need to be enclosed in 
> `--whole-archive` in order to be included in the link.
Right, good point.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D118171/new/

https://reviews.llvm.org/D118171

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to