[PATCH] D24070: Setting fp trapping mode and denormal type

[Matt Arsenault via llvm-commits]

arsenm added inline comments.

================
Comment at: llvm/trunk/include/llvm/CodeGen/CommandFlags.h:158-159
@@ +157,4 @@
+
+cl::opt<llvm::FPDenormal::DenormalType>
+DenormalType("denormal-fp-math",
+          cl::desc("Select which denormal numbers the code is permitted to require"),
----------------
SjoerdMeijer wrote:
> arsenm wrote:
> > Should this be per-FP type? We currently use a subtarget feature for this but need to split between f16/f32/f64
> I now see how you are using it. We adopted the same approach as some other fp options (no NaNs, no infs, etc), which are also not per FP type. So to be honest, I don't know if they should. 
The other options are quite different from this. They are optimization hints that aren't required for correctness (and also are deprecated in favor of the per-instruction flags). The global denormal handling mode is different and required


Repository:
  rL LLVM

https://reviews.llvm.org/D24070