[PATCH] D57327: [ARM] Thumb2: ConstantMaterializationCost

[Sjoerd Meijer via Phabricator via llvm-commits]

SjoerdMeijer added inline comments.


================
Comment at: lib/Target/ARM/ARMISelDAGToDAG.cpp:454
     if (Subtarget->hasV6T2Ops() &&
-        (Val <= 0xffff || ARM_AM::getT2SOImmValSplatVal(Val) != -1))
+        (Val <= 0xffff || ARM_AM::getT2SOImmVal(Val) != -1 ||  // MOV
+         ARM_AM::getT2SOImmVal(~Val) != -1))                   // MVN
----------------
efriedma wrote:
> I don't think the change from getT2SOImmValSplatVal to getT2SOImmVal makes much practical difference; isThumbImmShiftedVal covers the same cases.  But I guess it's more clear, at least.
> 
> Maybe fix the MOV/MVN/MOVW comments so they're each on the same line as the corresponding check?
Forgot to mention that, but indeed, I found there was no practical difference, but thought it is clearer.


================
Comment at: test/CodeGen/ARM/shifter_operand.ll:262
+
+define hidden i32 @no_litpool() local_unnamed_addr #0 {
+; CHECK-LABEL:  no_litpool:
----------------
efriedma wrote:
> You can put the attributes inline here (instead of "#0", just write "optsize minsize").
Thanks, didn't know that!


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

https://reviews.llvm.org/D57327