Skip to content

Commit ea6efe9

Browse files
jgu222igcbot
authored andcommitted
[Autobackout][FuncReg]Revert of change: 4e3654a
minor change Fix comments. Don't call clear() twice as releaseMemory() will do it. Make sure return true if ldstCombine does store combining.
1 parent c42d809 commit ea6efe9

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed

IGC/Compiler/CISACodeGen/MemOpt.cpp

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2779,14 +2779,17 @@ bool LdStCombine::runOnFunction(Function& F)
27792779
m_symEval.setDataLayout(m_DL);
27802780

27812781
// i64Emu: mimic Emu64Ops's enabling condition. Seems conservative
2782-
// but can be improved in the future if needed.
2782+
// and be improved in the future if needed.
27832783
m_hasI64Emu = (m_CGC->platform.need64BitEmulation() &&
27842784
(IGC_GET_FLAG_VALUE(Enable64BitEmulation) ||
27852785
IGC_GET_FLAG_VALUE(Enable64BitEmulationOnSelectedPlatform)));
27862786

27872787
combineStores(F);
27882788

27892789
bool changed = (m_hasLoadCombined || m_hasStoreCombined);
2790+
2791+
clear();
2792+
27902793
return changed;
27912794
}
27922795

@@ -3934,8 +3937,6 @@ void LdStCombine::createCombinedStores(Function& F)
39343937
// Delete stores that have been combined.
39353938
eraseDeadInsts();
39363939

3937-
m_hasStoreCombined = (!m_bundles.empty());
3938-
39393940
m_bundles.clear();
39403941
}
39413942

0 commit comments

Comments
 (0)