This is an archive of the discontinued LLVM Phabricator instance.

[SelectionDAGBuilder] Add SPF_NABS support to visitSelect
ClosedPublic

Authored by craig.topper on Nov 25 2020, 10:57 AM.

Details

Summary

We currently don't match this which limits the effectiveness of D91120 until InstCombine starts canonicalizing to llvm.abs. This should be easy to remove if/when we remove the SPF_ABS handling.

Diff Detail

Event Timeline

craig.topper created this revision.Nov 25 2020, 10:57 AM
craig.topper requested review of this revision.Nov 25 2020, 10:57 AM
RKSimon accepted this revision.Nov 25 2020, 2:09 PM

LGTM

This revision is now accepted and ready to land.Nov 25 2020, 2:09 PM