[PATCH] D67591: [ARM] A predicate cast of a predicate cast is a predicate cast

Sam Parker via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 16 07:28:46 PDT 2019


samparker accepted this revision.
samparker added a comment.
This revision is now accepted and ready to land.

Good point, the copies probably would do a good job of constraining the scheduler!



================
Comment at: llvm/lib/Target/ARM/ARMISelLowering.cpp:12641
+    return DCI.DAG.getNode(ARMISD::PREDICATE_CAST, dl,
+                           Op->getOperand(0).getValueType(), Op->getOperand(0));
+  }
----------------
Shouldn't this be casting to VT?


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

https://reviews.llvm.org/D67591





More information about the llvm-commits mailing list