With the introduction of the native fneg instruction, the
fsub -0.0, %x idiom is obsolete. This patch makes LLVM
emit fneg instead of the idiom in all places.
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Unit Tests
Unit Tests
Event Timeline
| llvm/lib/Transforms/InstCombine/InstCombineSelect.cpp | ||
|---|---|---|
| 341 | What about this TODO? | |
| llvm/lib/Transforms/InstCombine/InstCombineSelect.cpp | ||
|---|---|---|
| 341 | Ok. So this seems to be a hack to update the FSub idiom on the fly. | |
| llvm/lib/Transforms/InstCombine/InstCombineSelect.cpp | ||
|---|---|---|
| 341 | I agree | |
Comment Actions
LGTM, with the TODO to be handled under a separate patch.
Thank you for taking care of this, Simon.
| llvm/lib/Transforms/InstCombine/InstCombineSelect.cpp | ||
|---|---|---|
| 341 | +1 | |
What about this TODO?