This is an archive of the discontinued LLVM Phabricator instance.

[X86] Update SSE/AVX integer MINMAX intrinsics to emit llvm.smax.* etc. (PR46851)
ClosedPublic

Authored by RKSimon on Sep 14 2020, 3:15 AM.

Details

Summary

We're now getting close to having the necessary analysis/combines etc. for the new generic llvm smax/smin/umax/umin intrinsics.

This patch updates the SSE/AVX integer MINMAX intrinsics to emit the generic equivalents instead of the icmp+select code pattern.

Diff Detail

Event Timeline

RKSimon created this revision.Sep 14 2020, 3:15 AM
Herald added projects: Restricted Project, Restricted Project. · View Herald TranscriptSep 14 2020, 3:15 AM
RKSimon requested review of this revision.Sep 14 2020, 3:15 AM
nikic accepted this revision.Sep 14 2020, 12:58 PM

LGTM. I think we have enough to start using them for vector intrinsics.

This revision is now accepted and ready to land.Sep 14 2020, 12:58 PM