This is an archive of the discontinued LLVM Phabricator instance.

Differential D84880

[AIX] Temporarily disable IncrementalProcessingTest partially
ClosedPublic

Authored by Xiangling_L on Jul 29 2020, 11:40 AM.

Details

Reviewers
daltenty
jasonliu
stevewan
hubert.reinterpretcast
Commits
rG4e6176fd912a: [AIX] Temporarily disable IncrementalProcessingTest partially

Diff Detail

Event Timeline

Xiangling_L created this revision.Jul 29 2020, 11:40 AM
Herald added a project: Restricted Project. · View Herald TranscriptJul 29 2020, 11:40 AM
Herald added a subscriber: cfe-commits. · View Herald Transcript
Xiangling_L requested review of this revision.Jul 29 2020, 11:40 AM
Harbormaster completed remote builds in B66264: Diff 281691.Jul 29 2020, 11:40 AM
daltenty accepted this revision.Jul 29 2020, 12:37 PM

LGTM, thanks!

This revision is now accepted and ready to land.Jul 29 2020, 12:37 PM
This revision was landed with ongoing or failed builds.Jul 30 2020, 7:42 AM
Closed by commit rG4e6176fd912a: [AIX] Temporarily disable IncrementalProcessingTest partially (authored by Xiangling_L). · Explain Why
This revision was automatically updated to reflect the committed changes.
Xiangling_L added a commit: rG4e6176fd912a: [AIX] Temporarily disable IncrementalProcessingTest partially.

Revision Contents

PathSize
clang/
unittests/
CodeGen/
5 lines

Diff 281691

clang/unittests/CodeGen/IncrementalProcessingTest.cpp

Show First 20 Lines • Show All 153 Lines • ▼ Show 20 LinesTEST(IncrementalProcessing, EmitCXXGlobalInitFunc) {
ASSERT_TRUE(M[1]->getFunction("funcForProg1")); ASSERT_TRUE(M[1]->getFunction("funcForProg1"));
M[2] = IncrementalParseAST(compiler, P, *CG, TestProgram2); M[2] = IncrementalParseAST(compiler, P, *CG, TestProgram2);
ASSERT_TRUE(M[2]); ASSERT_TRUE(M[2]);
ASSERT_TRUE(M[2]->getFunction("funcForProg2")); ASSERT_TRUE(M[2]->getFunction("funcForProg2"));
// First code should not end up in second module: // First code should not end up in second module:
ASSERT_FALSE(M[2]->getFunction("funcForProg1")); ASSERT_FALSE(M[2]->getFunction("funcForProg1"));
// TODO: Remove this after the static initialization frontend implementation
// is recovered on AIX.
if (compiler.getTarget().getTriple().isOSAIX())
return;
// Make sure global inits exist and are unique: // Make sure global inits exist and are unique:
const Function* GlobalInit1 = getGlobalInit(*M[1]); const Function* GlobalInit1 = getGlobalInit(*M[1]);
ASSERT_TRUE(GlobalInit1); ASSERT_TRUE(GlobalInit1);
const Function* GlobalInit2 = getGlobalInit(*M[2]); const Function* GlobalInit2 = getGlobalInit(*M[2]);
ASSERT_TRUE(GlobalInit2); ASSERT_TRUE(GlobalInit2);
ASSERT_FALSE(GlobalInit1->getName() == GlobalInit2->getName()); ASSERT_FALSE(GlobalInit1->getName() == GlobalInit2->getName());
} }
} // end anonymous namespace } // end anonymous namespace