[PATCH] D57327: [ARM] Thumb2: ConstantMaterializationCost

[Sjoerd Meijer via Phabricator via llvm-commits]

SjoerdMeijer marked an inline comment as done.
SjoerdMeijer added inline comments.


================
Comment at: lib/Target/ARM/ARMISelDAGToDAG.cpp:454
     if (Subtarget->hasV6T2Ops() &&
-        (Val <= 0xffff || ARM_AM::getT2SOImmValSplatVal(Val) != -1))
-      return 1; // MOVW
+        (Val <= 0xffff || ARM_AM::getT2SOImmVal(Val) != -1 || // MOV + MOVW
+         ARM_AM::getT2SOImmVal(~Val) != -1))                  // MVN
----------------
efriedma wrote:
> The "+" here is misleading (especially given other nearby comments); either replace it with a "/", or put the checks on separate lines.
Thanks, will do before committing.


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

https://reviews.llvm.org/D57327