[PATCH] D47323: FastMathFlags: Make it easier to unset individual ones.

[Nicola Zaghen via Phabricator via llvm-commits]

Nicola added inline comments.


================
Comment at: include/llvm/IR/Operator.h:210-214
+  void setAllowReassoc(bool B = true) {
+    Flags = (Flags & ~AllowReassoc) | B * AllowReassoc;
+  }
+  void setNoNaNs(bool B = true) { Flags = (Flags & ~NoNaNs) | B * NoNaNs; }
+  void setNoInfs(bool B = true) { Flags = (Flags & ~NoInfs) | B * NoInfs; }
----------------
spatel wrote:
> Run clang-format on this? 
> 
> Would be easier to see that these are all the same basic logic if they all had the function body on a separate line.
I did, that's how it ended up like that :(

I'll make it look "nice" again and submit. 


Repository:
  rL LLVM

https://reviews.llvm.org/D47323