Page MenuHomePhabricator
Differential D92169

[Hexagon] Add HVX support for ISD::SMAX/SMIN/UMAX/UMIN instead of custom dag patterns
ClosedPublic
Actions

Authored by RKSimon on Nov 26 2020, 3:39 AM.

Details

Reviewers
kparzysz
Commits
rGc4628460b74b: [Hexagon] Add HVX support for ISD::SMAX/SMIN/UMAX/UMIN instead of custom dag…
Summary

Followup to D92112 now that I've learnt about hvx type splitting.

This is some necessary cleanup work for min/max ops to eventually help us move the add/sub sat patterns into DAGCombine - D91876.

Diff Detail

Event Timeline

RKSimon created this revision.Nov 26 2020, 3:39 AM
Herald added a project: Restricted Project. · View Herald TranscriptNov 26 2020, 3:39 AM
Herald added subscribers: steven.zhang, hiraditya. · View Herald Transcript
RKSimon requested review of this revision.Nov 26 2020, 3:39 AM
Harbormaster completed remote builds in B80207: Diff 307815.Nov 26 2020, 4:19 AM
kparzysz accepted this revision.Nov 27 2020, 6:52 AM

LGTM, Thanks!

This revision is now accepted and ready to land.Nov 27 2020, 6:52 AM
This revision was landed with ongoing or failed builds.Nov 27 2020, 7:46 AM
Closed by commit rGc4628460b74b: [Hexagon] Add HVX support for ISD::SMAX/SMIN/UMAX/UMIN instead of custom dag… (authored by RKSimon). · Explain Why
This revision was automatically updated to reflect the committed changes.
RKSimon added a commit: rGc4628460b74b: [Hexagon] Add HVX support for ISD::SMAX/SMIN/UMAX/UMIN instead of custom dag….

No Actions

Showing all changes because you have never taken an action on this revision.

Revision Contents
Changeset List

PathSize
llvm/
lib/
Target/
Hexagon/
18 lines
26 lines
test/
CodeGen/
Hexagon/
autohvx/
40 lines
40 lines

Diff 308057

View Options

llvm/lib/Target/Hexagon/HexagonISelLoweringHVX.cpp

Loading...
View Options

llvm/lib/Target/Hexagon/HexagonPatternsHVX.td

Loading...
View Options

llvm/test/CodeGen/Hexagon/autohvx/minmax-128b.ll

Loading...
View Options

llvm/test/CodeGen/Hexagon/autohvx/minmax-64b.ll

Loading...