Page MenuHomePhabricator
Differential D145458

[InstSimplify] Add simplifications for `min/max(Abs(X), X)`
Needs RevisionPublic
Actions

Authored by goldstein.w.n on Mon, Mar 6, 6:07 PM.

Details

Reviewers
nikic
spatel
majnemer
Summary

smax(Abs(X, t/f), X) --> Abs(X)

umin(Abs(X, t/f), X) --> Abs(X)

smin(Abs(X, t/f), X) --> X

umax(Abs(X, t/f), X) --> X

Diff Detail

Event Timeline

goldstein.w.n created this revision.Mon, Mar 6, 6:07 PM
Herald added a project: Restricted Project. · View Herald TranscriptMon, Mar 6, 6:07 PM
Herald added a subscriber: hiraditya. · View Herald Transcript
goldstein.w.n requested review of this revision.Mon, Mar 6, 6:07 PM
Herald added a project: Restricted Project. · View Herald TranscriptMon, Mar 6, 6:07 PM
Herald added a subscriber: llvm-commits. · View Herald Transcript
Harbormaster completed remote builds in B217773: Diff 502884.Mon, Mar 6, 6:08 PM
goldstein.w.n added reviewers: nikic, spatel, majnemer.Mon, Mar 6, 6:08 PM
Herald added a subscriber: StephenFan. · View Herald TranscriptMon, Mar 6, 6:08 PM
goldstein.w.n added a parent revision: D145457: [InstSimplify] Add simplifications for `umin/umax(uadd/usub.sat(X, Y), X)`.Mon, Mar 6, 6:08 PM
nikic requested changes to this revision.Thu, Mar 9, 7:16 AM

Same here, you want folds along the lines of https://alive2.llvm.org/ce/z/4fWUpA.

This revision now requires changes to proceed.Thu, Mar 9, 7:16 AM

Revision Contents
Changeset List

PathSize
llvm/
lib/
Analysis/
6 lines
test/
Transforms/
InstSimplify/
14 lines

Diff 502884

View Options

llvm/lib/Analysis/InstructionSimplify.cpp

Loading...
View Options

llvm/test/Transforms/InstSimplify/minmax-intrin.ll

Loading...