[PATCH] D73287: AMDGPU/GlobalISel: Combine FMIN_LEGACY/FMAX_LEGACY

[Matt Arsenault via Phabricator via llvm-commits]

arsenm created this revision.
arsenm added reviewers: nhaehnle, kerbowa.
Herald added subscribers: Petar.Avramovic, hiraditya, t-tye, tpr, dstuttard, rovka, yaxunl, wdng, jvesely, kzhuravl.
Herald added a project: LLVM.

Try out using combine definition rules.

     

This really should be a post-legalizer combine, but the combiner pass
is currently pre-legalize. Most of the target combines are really
 post-legalize, so we should probably move the pass.


https://reviews.llvm.org/D73287

Files:
  llvm/lib/Target/AMDGPU/AMDGPUCombine.td
  llvm/lib/Target/AMDGPU/AMDGPUGISel.td
  llvm/lib/Target/AMDGPU/AMDGPUPreLegalizerCombiner.cpp
  llvm/lib/Target/AMDGPU/AMDGPURegisterBankInfo.cpp
  llvm/lib/Target/AMDGPU/SIInstructions.td
  llvm/test/CodeGen/AMDGPU/GlobalISel/fmax_legacy.ll
  llvm/test/CodeGen/AMDGPU/GlobalISel/fmin_legacy.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D73287.239978.patch
Type: text/x-patch
Size: 30933 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200123/f78c4f14/attachment.bin>