[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