[PATCH] D110142: [clang][Driver] Correct runtime path for Arm hard float targets

Tue Sep 21 11:01:21 PDT 2021

phosek added a comment.

In D110142#3013138 <https://reviews.llvm.org/D110142#3013138>, @MaskRay wrote:

> Do you know why `--target=armv8l-unknown-linux-gnueabihf` picks `armhf-unknown-linux-gnueabihf` instead of `armv8l-unknown-linux-gnueabihf`?
> (For new tests, prefer `--target=` to `-target `; space separated driver options are not the convention.)
>
> The direction the Clang driver should move to is less magic. See D109727 <https://reviews.llvm.org/D109727>

I agree with @MaskRay, we discussed this when updating the Clang multiarch layout and the decision was to use normalized LLVM triples to avoid additional translations as is done in this change.

It also seems like `armhf-unknown-linux-gnueabihf` duplicates the `hf` bit (there's both `armhf` and `gnueabihf`) which seems unnecessary.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D110142