Page MenuHomePhabricator
Differential D139895

InstCombine: Fold is.fpclass (fneg x) into the test mask
ClosedPublic
Actions

Authored by arsenm on Dec 12 2022, 4:53 PM.

Details

Reviewers
sepavloff
andrew.w.kaylor
jcranmer-intel
foad
cameron.mcinally

Diff Detail

Event Timeline

arsenm created this revision.Dec 12 2022, 4:53 PM
Herald added a project: Restricted Project. · View Herald TranscriptDec 12 2022, 4:53 PM
Herald added a subscriber: hiraditya. · View Herald Transcript
arsenm requested review of this revision.Dec 12 2022, 4:53 PM
Herald added a project: Restricted Project. · View Herald TranscriptDec 12 2022, 4:53 PM
Herald added a subscriber: wdng. · View Herald Transcript
Harbormaster completed remote builds in B202720: Diff 482310.Dec 12 2022, 4:54 PM
foad accepted this revision.Dec 13 2022, 5:48 AM

LGTM. Could do the same kind of thing with abs.

This revision is now accepted and ready to land.Dec 13 2022, 5:48 AM
foad added a comment.Dec 13 2022, 6:02 AM
In D139895#3991686, @foad wrote:

Could do the same kind of thing with abs.

I've just seen D139903!

arsenm added a parent revision: D137811: InstCombine: Perform basic isnan combines on llvm.is.fpclass.Dec 16 2022, 7:11 AM
jcranmer-intel accepted this revision.Dec 20 2022, 8:35 AM
arsenm closed this revision.Feb 5 2023, 5:10 AM

a520851d1286edee59449f2d90bd04a982e54f8d

Herald added a subscriber: StephenFan. · View Herald TranscriptFeb 5 2023, 5:10 AM

Revision Contents
Changeset List

PathSize
llvm/
lib/
Transforms/
InstCombine/
25 lines
test/
Transforms/
InstCombine/
151 lines

Diff 482310

View Options

llvm/lib/Transforms/InstCombine/InstCombineCalls.cpp

Loading...
View Options

llvm/test/Transforms/InstCombine/is_fpclass.ll

Loading...