MaskRay added a comment.
Herald added a reviewer: ctetreau.

In D110663#3029577 <https://reviews.llvm.org/D110663#3029577>, @phosek wrote:

> This is going to break Fuchsia as implemented. We assume that 
> `{x86_64,aarch64}-unknown-fuchsia` and `{x86_64,aarch64}-fuchsia` behave 
> identically and developers are allowed to use whichever spelling they prefer. 
> With this change that's no longer true. This may break other platforms that 
> make similar assumptions.

Are Fuchsia developers specifying `--target=` explicitly? I think that's the 
direction we don't want to support.

From my understanding of D110900 <https://reviews.llvm.org/D110900>, if a Linux 
distro prefers `*-linux` to `*-linux-gnu`, we should drop `--target=` 
normalization to make this available.

I can special case Fuchsia to use the normalized triple, but that's code that I 
want to avoid.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D110663

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

Reply via email to