[PATCH] D155201: [Clang][AArch64] Introduce CLANG_AARCH64_DEFAULT_CPU

Sjoerd Meijer via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 13 07:15:50 PDT 2023


SjoerdMeijer added a comment.

In D155201#4497585 <https://reviews.llvm.org/D155201#4497585>, @tschuett wrote:

> Clang configuration files are insufficient for this task?
> https://clang.llvm.org/docs/UsersManual.html#configuration-files

I don't know, but will look into it.
Ideally, we could change this:

  CHECK:  "-cc1"{{.*}} "-triple" "aarch64{{.*}}" "-target-cpu" "generic"

into:

  CHECK:  "-cc1"{{.*}} "-triple" "aarch64{{.*}}" "-target-cpu" "$(CLANG_AARCH64_DEFAULT_CPU)"

So I feel that this is more of a lit think. Lit has to understand the value of CLANG_AARCH64_DEFAULT_CPU.


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

https://reviews.llvm.org/D155201



More information about the llvm-commits mailing list