[PATCH] D85257: [X86] Optimize getImpliedDisabledFeatures & getImpliedEnabledFeatures

[Craig Topper via Phabricator via llvm-commits]

craig.topper added a comment.

I should also mention that the code that reason clang is doing this is to check the ABI compatibility for vector arguments to produce a nice error message about feature mismatch. The backend is also able to generate an error for the same case. Its just not as nice as it won't tell you that avx512 is missing. So we're paying a high cost for nicer error message. The inline asm should have source location so I actually wonder if the backend could be made to print a better error message itself.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D85257