[llvm] [X86] Use RORX over SHR imm (PR #77964)

Shengchen Kan via llvm-commits llvm-commits at lists.llvm.org
Sun Jan 14 18:58:20 PST 2024


https://github.com/KanRobert requested changes to this pull request.

Sorry for request change on this. I have two concerns here
1.  Whether this change will bring benefits is questionable b/c
      a.  it introduces false dependency 
      b.  code size of RORX is longer than SHR
2.  it's probably not the best solution to do this by adding new pattern. This 
      optimization should occur at most when there is a user in the previously generated 
      EFLAGS. 

https://github.com/llvm/llvm-project/pull/77964


More information about the llvm-commits mailing list