This is an archive of the discontinued LLVM Phabricator instance.

Differential D5815

[mips] Remove redundant code in RetCC_MipsN.
ClosedPublic

Authored by dsanders on Oct 16 2014, 2:10 AM.

Details

Reviewers
vmedic
Summary

i32 is always promoted to i64 so it no longer makes sense to assign i32 to
registers.

Diff Detail

Event Timeline

dsanders updated this revision to Diff 14994.Oct 16 2014, 2:10 AM
dsanders retitled this revision from to [mips] Remove redundant code in RetCC_MipsN..
dsanders updated this object.
dsanders edited the test plan for this revision. (Show Details)
dsanders added a reviewer: vmedic.
dsanders added subscribers: theraven, Unknown Object (MLST).
vmedic accepted this revision.Nov 20 2014, 4:12 AM
vmedic edited edge metadata.

LGTM

This revision is now accepted and ready to land.Nov 20 2014, 4:12 AM
dsanders closed this revision.Dec 8 2014, 7:13 AM

Revision Contents

PathSize
lib/
Target/
Mips/
3 lines

Diff 14994

lib/Target/Mips/MipsCallingConv.td

Show First 20 Lines • Show All 135 Lines • ▼ Show 20 Linesdef RetCC_MipsN : CallingConv<[
// Aggregate returns are positioned at the lowest address in the slot for // Aggregate returns are positioned at the lowest address in the slot for
// both little and big-endian targets. When passing in registers, this // both little and big-endian targets. When passing in registers, this
// requires that big-endian targets shift the value into the upper bits. // requires that big-endian targets shift the value into the upper bits.
CCIfSubtarget<"isLittle()", CCIfSubtarget<"isLittle()",
CCIfType<[i8, i16, i32, i64], CCIfInReg<CCPromoteToType<i64>>>>, CCIfType<[i8, i16, i32, i64], CCIfInReg<CCPromoteToType<i64>>>>,
CCIfSubtargetNot<"isLittle()", CCIfSubtargetNot<"isLittle()",
CCIfType<[i8, i16, i32, i64], CCIfInReg<CCPromoteToUpperBitsInType<i64>>>>, CCIfType<[i8, i16, i32, i64], CCIfInReg<CCPromoteToUpperBitsInType<i64>>>>,
// i32 are returned in registers V0, V1
CCIfType<[i32], CCAssignToReg<[V0, V1]>>,
// i64 are returned in registers V0_64, V1_64 // i64 are returned in registers V0_64, V1_64
CCIfType<[i64], CCAssignToReg<[V0_64, V1_64]>>, CCIfType<[i64], CCAssignToReg<[V0_64, V1_64]>>,
// f32 are returned in registers F0, F2 // f32 are returned in registers F0, F2
CCIfType<[f32], CCAssignToReg<[F0, F2]>>, CCIfType<[f32], CCAssignToReg<[F0, F2]>>,
// f64 are returned in registers D0, D2 // f64 are returned in registers D0, D2
CCIfType<[f64], CCAssignToReg<[D0_64, D2_64]>> CCIfType<[f64], CCAssignToReg<[D0_64, D2_64]>>
▲ Show 20 Lines • Show All 169 Lines • Show Last 20 Lines