[PATCH] D53349: [VPlan] Changes to implement VPlan based predication for VPlan-native path.

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Jan 12 10:50:36 PST 2019


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

LGTM, thanks! Please wait a bit with committing, in case anybody else has additional comments.



================
Comment at: unittests/Transforms/Vectorize/VPlanPredicatorTest.cpp:214
+  EXPECT_EQ(Or->getOpcode(), Instruction::Or);
+  EXPECT_EQ(TSuccSucc->getPredicate(), Or);
+}
----------------
Please check the operands of the OR here as well.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D53349





More information about the llvm-commits mailing list