[PATCH] Jump thread llvm.expect
Philip Reames
listmail at philipreames.com
Mon Jan 26 10:19:28 PST 2015
Looks straight forward. Provided your confident of the answer to my inline question, LGTM.
================
Comment at: lib/Transforms/Scalar/JumpThreading.cpp:520
@@ -512,3 +519,3 @@
Value *LHS = PN->getIncomingValue(i);
Value *RHS = Cmp->getOperand(1)->DoPHITranslation(BB, PredBB);
----------------
Can the PHITranslation now fail with your change?
================
Comment at: test/Transforms/JumpThreading/expect.ll:10
@@ +9,3 @@
+
+define i64 @test(i1 %cond) {
+ br i1 %cond, label %T1, label %F1
----------------
Please use CHECK-LABEL so this file is easy to extend later.
Also, comment description at top of file should be clearer.
http://reviews.llvm.org/D7169
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
More information about the llvm-commits
mailing list