[llvm] [llvm-exegesis] Exclude loads/stores from aliasing instruction set (PR #156300)
Sjoerd Meijer via llvm-commits
llvm-commits at lists.llvm.org
Wed Sep 3 06:35:37 PDT 2025
sjoerdmeijer wrote:
> `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.
> [09c2839](https://github.com/llvm/llvm-project/commit/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.
Thanks for the suggestion, have move the checks to that place.
https://github.com/llvm/llvm-project/pull/156300
More information about the llvm-commits
mailing list