[PATCH] D81108: [AArch64] Fix ldst-opt of multiple disjunct subregs.
    Eli Friedman via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Fri Jun  5 17:54:44 PDT 2020
    
    
  
efriedma accepted this revision.
efriedma added a comment.
This revision is now accepted and ready to land.
LGTM with one minor question.
> I think the way we choose rename registers prevents using a register that is defined by the same instruction we are renaming. I can see if I can add a test case for that.
Did you ever come up with a testcase for this?
Repository:
  rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D81108/new/
https://reviews.llvm.org/D81108
    
    
More information about the llvm-commits
mailing list