This is an archive of the discontinued LLVM Phabricator instance.

[LoongArch] Return null SDValue by default in LowerOperation. NFC
ClosedPublic

Authored by SixWeining on Aug 10 2022, 12:37 AM.

Diff Detail

Event Timeline

SixWeining created this revision.Aug 10 2022, 12:37 AM
Herald added a project: Restricted Project. · View Herald TranscriptAug 10 2022, 12:37 AM
SixWeining requested review of this revision.Aug 10 2022, 12:37 AM
Herald added a project: Restricted Project. · View Herald TranscriptAug 10 2022, 12:37 AM
xen0n accepted this revision.Aug 10 2022, 1:54 AM

If tests continue to pass then this is good :-)

This revision is now accepted and ready to land.Aug 10 2022, 1:54 AM

Return SDValue() by default.

SixWeining retitled this revision from [LoongArch] Remove null LowerOperation to [LoongArch] Return null SDValue by default in LowerOperation. NFC.Aug 10 2022, 3:06 AM
SixWeining edited the summary of this revision. (Show Details)

If tests continue to pass then this is good :-)

Sorry. The test result shows that ISD::FP_TO_UINT, the return SDValue() should be kept.