[PATCH] D12007: [AMDGPU] Use the general SMAX/SMIN/UMAX/UMIN pattern matching and remove the AMDGPU implementation

Matt Arsenault via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 13 12:26:53 PDT 2015


arsenm accepted this revision.
arsenm added a comment.
This revision is now accepted and ready to land.

I think I tried to do this before, but hit a regression where a min/max pattern showed up during lowering since the generic min / max was only formed during SelectionDAGBuilder. I wasn't sure why min/max was only being matched so early. Was this fixed / is min/max matched in DAGCombiner now?


Repository:
  rL LLVM

http://reviews.llvm.org/D12007





More information about the llvm-commits mailing list