[PATCH] D20116: Add speculatable function attribute

Mehdi AMINI via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 18 15:30:40 PDT 2016


mehdi_amini added inline comments.

================
Comment at: docs/LangRef.rst:1521
@@ -1516,1 +1520,3 @@
+    particular exection path the number of calls to this function will not be
+    externally observable.
 ``ssp``
----------------
"does not have any effects besides calculating its result" and "speculatable is not enough to conclude that [...] the number of calls to this function will not be externally observable." seem contradictory to me.

(Also you have a typo with `exection` instead of `execution`)


https://reviews.llvm.org/D20116





More information about the llvm-commits mailing list