Now all the divisions should be complete, although we should fix
emitting the entire common part for div/rem when you use both.
Details
Details
Diff Detail
Diff Detail
Event Timeline
| llvm/lib/Target/AMDGPU/AMDGPULegalizerInfo.cpp | ||
|---|---|---|
| 2777 | I'd prefer to put an un-negated condition here, given that there's an "else". | |
| 2851–2852 | Unrelated but it seems a bit silly to duplicate these lines from 2847-2848. | |
| llvm/lib/Target/AMDGPU/AMDGPULegalizerInfo.h | ||
| 118 | Parameter names Num and Den don't match the definition. | |
Parameter names Num and Den don't match the definition.