[PATCH] D27028: Add intrinsics for constrained floating point operations
Matt Arsenault via llvm-commits
llvm-commits at lists.llvm.org
Wed Nov 23 13:22:14 PST 2016
arsenm added a comment.
In https://reviews.llvm.org/D27028#604714, @andrew.w.kaylor wrote:
> In https://reviews.llvm.org/D27028#604699, @arsenm wrote:
>
> > It's not handled in general llvm. We currently have a subtarget feature for f32/f64 denormal support in the default rounding mode
>
>
> It looks like several sub-targets have some kind of denormal support, but I didn't look closely enough to see what each of them is doing. That's kind of why I would prefer to defer the issue...because I haven't thought about it enough to know that I'd be implementing it correctly.
>
> So to get back to Hal's question, is this needed at a per-instruction level? I understand that you want to select instructions differently based on this setting, but is attaching it to the instruction just a convenient way to get the information to the ISel or can this change across different scopes?
It's the same as the rounding mode and be per-instruction
Repository:
rL LLVM
https://reviews.llvm.org/D27028
More information about the llvm-commits
mailing list