[PATCH] D153430: [Clang][Driver] Warn on invalid Arm or AArch64 baremetal target triple

Thu Jun 22 05:57:59 PDT 2023

peter.smith added a comment.

My initial reactions to seeing Invalid target triple aarch64-none-eabi; try aarch64-none-elf [*] were:

- Why is it invalid?
- I assumed it was an error message, and was about to write a comment until I saw it was a warning.

[X] now did you mean (thanks for making that change).

An alternative suggestion:

  <input triple> selects the fallback GCC toolchain driver; did you mean <proposed triple>

This isn't something I'm going to die on a hill for. Perhaps add some more reviewers to get some opinions, I'm happy to go with the consensus.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D153430