[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