[PATCH] D31276: Add #pragma clang fast_math
Adam Nemet via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Mar 23 09:47:09 PDT 2017
anemet added inline comments.
================
Comment at: docs/LanguageExtensions.rst:2321
+specified for a section of the source code. This pragma can only appear at
+file scope or at the start of a compound statement. When using within a
+compound statement, the pragma is active within the scope of the compound
----------------
aaron.ballman wrote:
> By "start of a compound statement", does that include or exclude things like comments? e.g., is this valid?
> ```
> {
> // Disable clang's fast-math
> #pragma clang fast_math contract_fast(off)
> }
> ```
Excluding comments. Added the words.
FTR, there is also test coverage for this already in the patch.
https://reviews.llvm.org/D31276
More information about the cfe-commits
mailing list