[PATCH] D22413: [ValueTracking] Teach isSafeToSpeculativelyExecute() about the speculatable attribute

[Matt Arsenault via llvm-commits]

arsenm added a comment.

Doesn't the SpeculativeExecution already considered the TTI cost for this anyway?


================
Comment at: lib/Analysis/ValueTracking.cpp:3120-3121
@@ -3119,2 +3119,4 @@
   }
   case Instruction::Call: {
+    const CallInst *CI = cast<CallInst>(Inst);
+    const Function *Callee = CI->getCalledFunction();
----------------
Is invoke allowed to be speculatable?


https://reviews.llvm.org/D22413