Skip to content
This repository was archived by the owner on Jan 23, 2023. It is now read-only.

Replace gtLclVarCommon. with AsLclVarCommon()-> #27207

Merged
merged 5 commits into from
Oct 21, 2019

Conversation

franksinankaya
Copy link

@franksinankaya franksinankaya changed the title find src/jit -type f -exec sed -i -e 's/gtLclVarCommon\./AsLclVarComm… Replace gtLclVarCommon. with AsLclVarCommon()-> Oct 15, 2019
@franksinankaya franksinankaya force-pushed the frkaya/gtLclVarCommon branch 2 times, most recently from 62f3185 to bcb3953 Compare October 17, 2019 01:50
Copy link

@sandreenko sandreenko left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, ignore the notes, but please delete the extra symbol in the license header.

@@ -3655,12 +3656,12 @@ void Compiler::optUnrollLoops()

/* Make sure everything looks ok */
if ((init->gtOper != GT_ASG) || (init->AsOp()->gtOp1->gtOper != GT_LCL_VAR) ||
(init->AsOp()->gtOp1->gtLclVarCommon.GetLclNum() != lvar) || (init->AsOp()->gtOp2->gtOper != GT_CNS_INT) ||
(init->AsOp()->gtOp2->AsIntCon()->gtIconVal != lbeg) ||
(init->AsOp()->gtOp1->AsLclVarCommon()->GetLclNum() != lvar) ||

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

note for me.

@@ -3727,7 +3727,8 @@ GenTreeCall* Compiler::fgMorphArgs(GenTreeCall* call)
assert(argEntry->structDesc.passedInRegisters);
if (lclVar != nullptr)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

note.

@@ -2202,7 +2202,7 @@ class CSE_Heuristic
cse = m_pCompiler->gtNewLclvNode(cseLclVarNum, cseLclVarTyp);

// Assign the ssa num for the use. Note it may be the reserved num.
cse->gtLclVarCommon.SetSsaNum(cseSsaNum);
cse->AsLclVarCommon()->SetSsaNum(cseSsaNum);

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

note

@@ -16435,7 +16436,8 @@ bool Compiler::impReturnInstruction(BasicBlock* block, int prefixFlags, OPCODE&
// Some other block(s) have seen the CEE_RET first.
// Better they spilled to the same temp.
assert(impInlineInfo->retExpr->gtOper == GT_LCL_VAR);

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

note

@@ -3075,8 +3076,8 @@ GenTree* Compiler::impInitializeArrayIntrinsic(CORINFO_SIG_INFO* sig)
//
GenTree* arrayAssignment = impLastStmt->GetRootNode();
if ((arrayAssignment->gtOper != GT_ASG) || (arrayAssignment->AsOp()->gtOp1->gtOper != GT_LCL_VAR) ||
(arrayLocalNode->gtOper != GT_LCL_VAR) ||
(arrayAssignment->AsOp()->gtOp1->gtLclVarCommon.GetLclNum() != arrayLocalNode->gtLclVarCommon.GetLclNum()))
(arrayLocalNode->gtOper != GT_LCL_VAR) || (arrayAssignment->AsOp()->gtOp1->AsLclVarCommon()->GetLclNum() !=

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

note

@sandreenko sandreenko merged commit 5e231b5 into dotnet:master Oct 21, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants