This is an archive of the discontinued LLVM Phabricator instance.

Differential D119102

NFC formating
AbandonedPublic

Authored by pravinjagtap on Feb 6 2022, 8:26 PM.

Details

Reviewers
sameerds

Diff Detail

Event Timeline

pravinjagtap created this revision.Feb 6 2022, 8:26 PM
Herald added subscribers: ormris, hiraditya. · View Herald TranscriptFeb 6 2022, 8:26 PM
pravinjagtap requested review of this revision.Feb 6 2022, 8:26 PM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 6 2022, 8:26 PM
Herald added a subscriber: llvm-commits. · View Herald Transcript
Harbormaster completed remote builds in B147872: Diff 406317.Feb 6 2022, 8:27 PM
pravinjagtap abandoned this revision.Feb 6 2022, 8:27 PM

Revision Contents

PathSize
llvm/
lib/
Transforms/
IPO/
2 lines

Diff 406317

llvm/lib/Transforms/IPO/ArgumentPromotion.cpp

Show First 20 Lines • Show All 192 Lines • ▼ Show 20 Lines if (ByValArgsToTransform.count(&*I)) {
if (LoadInst *L = dyn_cast<LoadInst>(UI)) if (LoadInst *L = dyn_cast<LoadInst>(UI))
OrigLoad = L; OrigLoad = L;
else else
// Take any load, we will use it only to update Alias Analysis // Take any load, we will use it only to update Alias Analysis
OrigLoad = cast<LoadInst>(UI->user_back()); OrigLoad = cast<LoadInst>(UI->user_back());
OriginalLoads[std::make_pair(&*I, Indices)] = OrigLoad; OriginalLoads[std::make_pair(&*I, Indices)] = OrigLoad;
// Preserve inbounds information of GEP so that it can be used in callers of current function // Preserve inbounds information of GEP so that it can be used in callers of current function
const GEPOperator *GEPOp = dyn_cast_or_null<GEPOperator>(UI); const GEPOperator *GEPOp = dyn_cast_or_null<GEPOperator>(UI);
if(GEPOp){ if(GEPOp) {
// Indices are unique in GEP since all the dead GEPs are already removed // Indices are unique in GEP since all the dead GEPs are already removed
IsArgGepInBound[std::make_pair(&*I, Indices)] = GEPOp->isInBounds(); IsArgGepInBound[std::make_pair(&*I, Indices)] = GEPOp->isInBounds();
} }
} }
// Add a parameter to the function for each element passed in. // Add a parameter to the function for each element passed in.
for (const auto &ArgIndex : ArgIndices) { for (const auto &ArgIndex : ArgIndices) {
// not allowed to dereference ->begin() if size() is 0 // not allowed to dereference ->begin() if size() is 0
▲ Show 20 Lines • Show All 996 Lines • Show Last 20 Lines