phosek added a comment.

In D151186#4369203 <https://reviews.llvm.org/D151186#4369203>, @MaskRay wrote:

> In D151186#4369181 <https://reviews.llvm.org/D151186#4369181>, @phosek wrote:
>
>> That alias is already recognized by the GNU driver. I'm sharing the 
>> implementation between the two drivers. Would you prefer to reject `-nopie` 
>> when targeting Fuchsia?
>
> Hmm, handling `OPT_nopie` in `clang/lib/Driver/ToolChains/Gnu.cpp` looks like 
> a mistake. We should reject it for Gnu.cpp...

I can send a separate change for that.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D151186

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

Reply via email to