[PATCH] D84100: [ARM] Optimize immediate selection

[Sam Parker via Phabricator via llvm-commits]

samparker added a comment.

I think the fact that you're having to make some Thumb2 changes to get around how the code is organised, suggests that the code should be reorganised instead. I'd suggest:

- Pre-commit the two-part-imm.ll test.
- Split the arm and thumb paths in an NFC patch, refactoring common code.
- Then make your arm changes.


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

https://reviews.llvm.org/D84100