This is an archive of the discontinued LLVM Phabricator instance.

[NFC] Remove redundant setOperationAction.
ClosedPublic

Authored by FreddyYe on Sep 22 2021, 7:20 PM.

Details

Summary

[FROUND,FROUNDEVEN][f32, f64, f128] are set Expand twice.

Diff Detail

Event Timeline

FreddyYe created this revision.Sep 22 2021, 7:20 PM
FreddyYe requested review of this revision.Sep 22 2021, 7:20 PM
Herald added a project: Restricted Project. · View Herald TranscriptSep 22 2021, 7:20 PM
FreddyYe updated this revision to Diff 374421.Sep 22 2021, 7:23 PM

Sorry for noisy update.

FreddyYe added inline comments.Sep 22 2021, 7:25 PM
llvm/lib/CodeGen/TargetLoweringBase.cpp
831

Redundant with here.

craig.topper accepted this revision.Sep 22 2021, 7:25 PM
craig.topper added a subscriber: craig.topper.

LGTM

This revision is now accepted and ready to land.Sep 22 2021, 7:25 PM
FreddyYe edited reviewers, added: pengfei, LuoYuanke; removed: craig.topper.Sep 22 2021, 7:25 PM
This revision now requires review to proceed.Sep 22 2021, 7:25 PM
This revision was not accepted when it landed; it landed in state Needs Review.Sep 22 2021, 7:29 PM
This revision was landed with ongoing or failed builds.
This revision was automatically updated to reflect the committed changes.