[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