[PATCH] D44216: [LangRef] make it clear that FP instructions do not have side effects

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 9 07:19:23 PST 2018


spatel added a comment.

In https://reviews.llvm.org/D44216#1032251, @lattner wrote:

> These changes look great, but I'd suggest explicitly mentioning that SNaN behavior somewhere, the relation of the instructions to the intrinsics etc.  Perhaps a section before Fast Math Flags?


Sure, I'll follow-up. I'm intentionally making minimal patches because that makes it less likely that I'll veer into the unknown. Also, any help with the phraseology is appreciated; I've only played a lawyer on TV. :)


https://reviews.llvm.org/D44216





More information about the llvm-commits mailing list