We don't need to figure out the weird extended rounding modes or
handle offsets to keep the lookup table in 64-bits.
Details
Diff Detail
Event Timeline
Code LGTM but I'm not well-versed in this kind of floating point magic so I will leave it up to someone that knows more to approve :)
llvm/lib/Target/AMDGPU/SIISelLowering.cpp | ||
---|---|---|
3611–3613 | nit: inline UseReducedTable? |
llvm/lib/Target/AMDGPU/SIISelLowering.cpp | ||
---|---|---|
3611–3613 | Then you lose the name-as-comment? |
Did you run adequate testing on this (OCL conformance maybe?) ? If so then I can approve it I think
llvm/lib/Target/AMDGPU/SIISelLowering.cpp | ||
---|---|---|
3628–3629 | Does that take a lot of effort to add? Why not add it now? |
llvm/lib/Target/AMDGPU/SIISelLowering.cpp | ||
---|---|---|
3618 | This is just AMDGPU::FltRoundToHWConversionTable & 0xFFFF isn't it? I'm not sure it's worth having all the boilerplate in SIModeRegisterDefaults.cpp to define another table. |
There's no wired up user of this
llvm/lib/Target/AMDGPU/SIISelLowering.cpp | ||
---|---|---|
3628–3629 | There are enough cases to think about in one patch as it is |
nit: inline UseReducedTable?