This is an archive of the discontinued LLVM Phabricator instance.

[RISCV][LegalizeDAG] Expand SETO and SETUO comparisons. Teach LegalizeDAG to expand SETUO expansion when UNE isn't legal.
ClosedPublic

Authored by craig.topper on Nov 23 2020, 11:56 PM.

Details

Summary

If SETUNE isn't legal, UO can use the NOT of the SETO expansion.

Removes some complex isel patterns. Most of the test changes are
from using XORI instead of SEQZ.

Diff Detail

Event Timeline

craig.topper created this revision.Nov 23 2020, 11:56 PM
Herald added a project: Restricted Project. · View Herald TranscriptNov 23 2020, 11:56 PM
craig.topper requested review of this revision.Nov 23 2020, 11:56 PM
asb accepted this revision.Dec 10 2020, 5:09 AM

It's great to be able to drop those ugly patterns - nice cleanup. Thanks.

This revision is now accepted and ready to land.Dec 10 2020, 5:09 AM
This revision was landed with ongoing or failed builds.Dec 10 2020, 9:16 AM
This revision was automatically updated to reflect the committed changes.