[BasicAA] Temporarily disable two tests
authorGeorge Burgess IV <george.burgess.iv@gmail.com>
Thu, 25 Jul 2019 06:53:59 +0000 (06:53 +0000)
committerGeorge Burgess IV <george.burgess.iv@gmail.com>
Thu, 25 Jul 2019 06:53:59 +0000 (06:53 +0000)
These tests are breaking three independent upstream buildbots (as well
downstream ones). These breakages have appeared mysteriously,
consistently, and during different revisions. Sadly, none of
{ASAN,TSAN,MSAN,UBSAN} flag anything, so the cause here is nonobvious.

Until we've figured this out, it seems best to disable these tests
entirely, so that the affected bots don't remain silent about any other,
unrelated failures.

Please see PR42719 for more information.

llvm-svn: 366986

llvm/unittests/Analysis/BasicAliasAnalysisTest.cpp

index f69d2d1..c08366e 100644 (file)
@@ -64,10 +64,13 @@ public:
       : M("BasicAATest", C), B(C), DL(DLString), TLI(TLII), F(nullptr) {}
 };
 
+// FIXME: Both of these are disabled at the moment due to strange buildbot
+// failures. Please see https://bugs.llvm.org/show_bug.cgi?id=42719
+
 // Check that a function arg can't trivially alias a global when we're accessing
 // >sizeof(global) bytes through that arg, unless the access size is just an
 // upper-bound.
-TEST_F(BasicAATest, AliasInstWithObjectOfImpreciseSize) {
+TEST_F(BasicAATest, DISABLED_AliasInstWithObjectOfImpreciseSize) {
   F = Function::Create(
       FunctionType::get(B.getVoidTy(), {B.getInt32Ty()->getPointerTo()}, false),
       GlobalValue::ExternalLinkage, "F", &M);
@@ -101,7 +104,7 @@ TEST_F(BasicAATest, AliasInstWithObjectOfImpreciseSize) {
 
 // Check that we fall back to MayAlias if we see an access of an entire object
 // that's just an upper-bound.
-TEST_F(BasicAATest, AliasInstWithFullObjectOfImpreciseSize) {
+TEST_F(BasicAATest, DISABLED_AliasInstWithFullObjectOfImpreciseSize) {
   F = Function::Create(
       FunctionType::get(B.getVoidTy(), {B.getInt64Ty()}, false),
       GlobalValue::ExternalLinkage, "F", &M);