[PATCH] D84100: [ARM] Optimize immediate selection

Sam Parker via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 11 03:20:30 PDT 2020


samparker added a comment.

If the logic isn't needed, then yes, I'd recommend removing. Currently I'm seeing logic added for Thumb2 opcodes so I would expect some changes and testing and if that isn't case, then I guess the changes aren't needed. Looks like the ORR and EOR paths also need to be tested?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D84100/new/

https://reviews.llvm.org/D84100



More information about the llvm-commits mailing list