[PATCH] D43451: [ARM] Mark -1 as cheap in xor's for thumb1

[Eli Friedman via Phabricator via llvm-commits]

efriedma added inline comments.


================
Comment at: llvm/trunk/lib/Target/ARM/ARMTargetTransformInfo.cpp:131
+  if (Opcode == Instruction::Xor) {
+    int64_t NegImm = Imm.getSExtValue();
+    if (NegImm == -1)
----------------
getSExtValue() will crash if Imm doesn't fit into an int64_t.  Maybe use `isAllOnesValue()`?


Repository:
  rL LLVM

https://reviews.llvm.org/D43451