[PATCH] Add a speculative execution pass

Jingyue Wu jingyue at google.com
Tue May 5 13:25:19 PDT 2015


================
Comment at: lib/Transforms/Scalar/SpeculativeExecution.cpp:60
@@ +59,3 @@
+
+static cl::opt<unsigned> SpecExecMaxNotHoisted(
+    "spec-exec-max-not-hoisted", cl::init(5), cl::Hidden,
----------------
jingyue wrote:
> Comment on why this threshold is necessary. My understanding is the NVIDIA driver only speculates a limited number of instructions. If too many instructions are left behind, the conditional basic block won't be executed in the SASS level. 
I meant "won't be speculatively executed".

http://reviews.llvm.org/D9360

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list