[llvm] r366986 - [BasicAA] Temporarily disable two tests

[George Burgess IV via llvm-commits]

Author: gbiv
Date: Wed Jul 24 23:53:59 2019
New Revision: 366986

URL: http://llvm.org/viewvc/llvm-project?rev=366986&view=rev
Log:
[BasicAA] Temporarily disable two tests

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.

Modified:
    llvm/trunk/unittests/Analysis/BasicAliasAnalysisTest.cpp

Modified: llvm/trunk/unittests/Analysis/BasicAliasAnalysisTest.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/unittests/Analysis/BasicAliasAnalysisTest.cpp?rev=366986&r1=366985&r2=366986&view=diff
==============================================================================
--- llvm/trunk/unittests/Analysis/BasicAliasAnalysisTest.cpp (original)
+++ llvm/trunk/unittests/Analysis/BasicAliasAnalysisTest.cpp Wed Jul 24 23:53:59 2019
@@ -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, AliasInstWithObjectO
 
 // 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);