[llvm] [llvm-exegesis] Exclude loads/stores from aliasing instruction set (PR #156300)
Aiden Grossman via llvm-commits
llvm-commits at lists.llvm.org
Mon Sep 1 11:01:48 PDT 2025
https://github.com/boomanaiden154 commented:
`hasMemoryOperands` (inside `llvm-exegesis` in https://github.com/llvm/llvm-project/blob/442e4ad92301882e8362edd64072f7cc6e36d141/llvm/tools/llvm-exegesis/lib/MCInstrDescView.cpp#L208) should be generic over different architectures.
09c2839c02a0eabf8851d81a44eba644ec683791 changed it from just requiring a memory operand to also requiring that the operand by explicit and a register. Not sure why it was changed. I'm thinking updating the condition there is the better option.
https://github.com/llvm/llvm-project/pull/156300
More information about the llvm-commits
mailing list