[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