[PATCH] D151696: [x86] Remove CPU_SPECIFIC* MACROs and add getCPUDispatchMangling

Mon Jun 26 06:48:58 PDT 2023

FreddyYe added a comment.

In D151696#4448573 <https://reviews.llvm.org/D151696#4448573>, @erichkeane wrote:

> I think this is OK, I have a slight fear we're losing a bit of the 'tune' functionality, but it is not impossible that we've never really cared about that.  One concern I have is that the list was used for the resolver function, but I don't see any test changes for that?  Are we properly filtering out the features list somehow?

Yes! Now there are no tests yet testing the resolver function influenced by the feature list. I'll add another pre-commit test to show this.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D151696