[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