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
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
