[PATCH] D24070: Setting fp trapping mode and denormal type
Matt Arsenault via llvm-commits
llvm-commits at lists.llvm.org
Thu Sep 1 09:12:11 PDT 2016
arsenm added inline comments.
Comment at: llvm/trunk/include/llvm/CodeGen/CommandFlags.h:158-159
@@ +157,4 @@
+ cl::desc("Select which denormal numbers the code is permitted to require"),
> 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
More information about the llvm-commits