[PATCH] D31496: Make -defsym a driver option
Salman Arif via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Apr 25 05:19:40 PDT 2017
salari01 abandoned this revision.
salari01 added a comment.
In https://reviews.llvm.org/D31496#721262, @rnk wrote:
> `-Wl,--defsym` is also a pretty common linker option. I'm hesitant to add this to the driver interface, since it seems like they could easily be confused.
I see. Was not aware of that linker option. Having looked it up now, I agree that the risk of confusion with the linker option outweighs the convenience of having this as a Driver option. I'll adandon this revision.
https://reviews.llvm.org/D31496
More information about the cfe-commits
mailing list