[PATCH] D79629: [Clang][Driver]Pass LLVM options to lld in case of LTO

2020-05-09 Thread bin via Phabricator via cfe-commits
bin.narwal added a comment. In D79629#2026974 , @MaskRay wrote: > `-mllvm,foobar` is for compilation (.c/.cc -> .o) > > `-Wl,-mllvm,foobar` for LTO options. For linking, `-mllvm,foobar` is not used > and thus warned. Right, that's reasonable design.

[PATCH] D79629: [Clang][Driver]Pass LLVM options to lld in case of LTO

2020-05-08 Thread Fangrui Song via Phabricator via cfe-commits
MaskRay requested changes to this revision. MaskRay added a comment. This revision now requires changes to proceed. `-mllvm,foobar` is for compilation (.c/.cc -> .o) `-Wl,-mllvm,foobar` for LTO options. For linking, `-mllvm,foobar` is not used and thus warned. Comment at:

[PATCH] D79629: [Clang][Driver]Pass LLVM options to lld in case of LTO

2020-05-08 Thread bin via Phabricator via cfe-commits
bin.narwal created this revision. bin.narwal added a reviewer: MaskRay. bin.narwal added a project: clang. Herald added subscribers: cfe-commits, dexonsmith, inglorion. Hi, When debugging ThinLTO with clang -mllvm -debug-only=function-import -flto=thin -fuse-ld=lld ..., it gives me below warning