[PATCH] D31276: Add #pragma clang fast_math

[Aaron Ballman via Phabricator via cfe-commits]

aaron.ballman added inline comments.


================
Comment at: docs/LanguageExtensions.rst:2319
+
+The ``#pragma clang fast_math`` allows floating-point fast-math options to be
+specified for a section of the source code.  This pragma can only appear at
----------------
Missing "pragma" in front of "allows".


================
Comment at: docs/LanguageExtensions.rst:2320
+The ``#pragma clang fast_math`` allows floating-point fast-math options to be
+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
----------------
Remove double spaces (applies elsewhere).


================
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
----------------
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)
}
```


================
Comment at: docs/LanguageExtensions.rst:2325
+
+Currently only FP contraction can be controlled with the pragma. ``#pragma
+clang fast_math contract_fast(on)`` enables contraction of a mulitply and an
----------------
Comma after Currently.


https://reviews.llvm.org/D31276