[PATCH] D156094: [AArch64] Ignore instructions not supported by CPU in AArch64SVESchedPseudoTest

Sander de Smalen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 24 07:59:22 PDT 2023


sdesmalen added a comment.

In D156094#4528143 <https://reviews.llvm.org/D156094#4528143>, @dmgreen wrote:

> Can you check this works in a Release build? If so it looks OK to me.

Thanks. I tried a Release build and it worked as expected.

> (It would be good if we had a more supported way of always querying an opcode for whether it was legal. I've wanted to use the same thing in llvm-exegesis recently).

Yes, I kind of expected there to be an interface for this already. Given there wasn't I wondered if this use-case was actually more niche than I thought. Feel free to move this out into a separate utility function if you want to reuse it for llvm-exegesis.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D156094



More information about the llvm-commits mailing list