[PATCH] D159250: [X86][RFC] Add new option `-m[no-]evex512` to disable ZMM and 64-bit mask instructions for AVX512 features

Phoebe Wang via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Sep 7 06:38:56 PDT 2023


pengfei added inline comments.


================
Comment at: clang/lib/CodeGen/Targets/X86.cpp:1493
                                  const llvm::StringMap<bool> &CalleeMap,
                                  QualType Ty, StringRef Feature,
                                  bool IsArgument) {
----------------
RKSimon wrote:
> Remove Feature argument and hardcode to "avx" now that it only has 1 (avx) caller?
We still have call to "avx512" at line 1525.
We have strict rule for AVX512-256, but should not change the legacy check.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D159250



More information about the cfe-commits mailing list