[PATCH] D52257: [Thumb1] Any imm of i8 type on Thumb1 should have cost of 1

Dave Green via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 21 01:21:00 PDT 2018


dmgreen added a comment.

Is this because we can just use a MOVS and wont have to fill in any higher bits? And MOVS's aren't trivially rematerialisable? And Thumb2/Arm are handled by getT2SOImmVal?

So, yeah, looks sensible to me, and this does seem to reduce codesize. But please clean up the testcase.


Repository:
  rL LLVM

https://reviews.llvm.org/D52257





More information about the llvm-commits mailing list