From 3b948587b26dfac366a02d8e44ca886b32cf6788 Mon Sep 17 00:00:00 2001 From: Chandler Carruth Date: Tue, 28 Jul 2015 11:11:11 +0000 Subject: [PATCH] [GMR] Teach GlobalsModRef to distinguish an important and safe case of no-alias with non-addr-taken globals: they cannot alias a captured pointer. If the non-global underlying object would have been a capture were it to alias the global, we can firmly conclude no-alias. It isn't reasonable for a transformation to introduce a capture in a way observable by an alias analysis. Consider, even if it were to temporarily capture one globals address into another global and then restore the other global afterward, there would be no way for the load in the alias query to observe that capture event correctly. If it observes it then the temporary capturing would have changed the meaning of the program, making it an invalid transformation. Even instrumentation passes or a pass which is synthesizing stores to global variables to expose race conditions in programs could not trigger this unless it queried the alias analysis infrastructure mid-transform, in which case it seems reasonable to return results from before the transform started. See the comments in the change for a more detailed outlining of the theory here. This should address the primary performance regression found when the non-conservatively-correct path of the alias query was disabled. Differential Revision: http://reviews.llvm.org/D11410 git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@243405 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/Analysis/IPA/GlobalsModRef.cpp | 46 ++++++++++++++ .../GlobalsModRef/nonescaping-noalias.ll | 62 +++++++++++++++++++ 2 files changed, 108 insertions(+) create mode 100644 test/Analysis/GlobalsModRef/nonescaping-noalias.ll diff --git a/lib/Analysis/IPA/GlobalsModRef.cpp b/lib/Analysis/IPA/GlobalsModRef.cpp index 18a7233075e..77fb41be8e2 100644 --- a/lib/Analysis/IPA/GlobalsModRef.cpp +++ b/lib/Analysis/IPA/GlobalsModRef.cpp @@ -701,6 +701,52 @@ AliasResult GlobalsModRef::alias(const MemoryLocation &LocA, if ((GV1 || GV2) && GV1 != GV2) return NoAlias; + // There are particular cases where we can conclude no-alias between + // a non-addr-taken global and some other underlying object. Specifically, + // a non-addr-taken global is known to not be escaped from any function. It + // is also incorrect for a transformation to introduce an escape of + // a global in a way that is observable when it was not there previously. + // One function being transformed to introduce an escape which could + // possibly be observed (via loading from a global or the return value for + // example) within another function is never safe. If the observation is + // made through non-atomic operations on different threads, it is + // a data-race and UB. If the observation is well defined, by being + // observed the transformation would have changed program behavior by + // introducing the observed escape, making it an invalid transform. + // + // This property does require that transformations which *temporarily* + // escape a global that was not previously escaped, prior to restoring + // it, cannot rely on the results of GMR::alias. This seems a reasonable + // restriction, although currently there is no way to enforce it. There is + // also no realistic optimization pass that would make this mistake. The + // closest example is a transformation pass which does reg2mem of SSA + // values but stores them into global variables temporarily before + // restoring the global variable's value. This could be useful to expose + // "benign" races for example. However, it seems reasonable to require that + // a pass which introduces escapes of global variables in this way to + // either not trust AA results while the escape is active, or to be forced + // to operate as a module pass that cannot co-exist with an alias analysis + // such as GMR. + if ((GV1 || GV2) && GV1 != GV2) { + const Value *UV = GV1 ? UV2 : UV1; + + // In order to know that the underlying object cannot alias the + // non-addr-taken global, we must know that it would have to be an + // escape. Thus if the underlying object is a function argument, a load + // from a global, or the return of a function, it cannot alias. + if (isa(UV) || isa(UV) || isa(UV)) { + // Arguments to functions or returns from functions are inherently + // escaping, so we can immediately classify those as not aliasing any + // non-addr-taken globals. + return NoAlias; + } else if (auto *LI = dyn_cast(UV)) { + // A pointer loaded from a global would have been captured, and we know + // that GV is non-addr-taken, so no alias. + if (isa(LI->getPointerOperand())) + return NoAlias; + } + } + // Otherwise if they are both derived from the same addr-taken global, we // can't know the two accesses don't overlap. } diff --git a/test/Analysis/GlobalsModRef/nonescaping-noalias.ll b/test/Analysis/GlobalsModRef/nonescaping-noalias.ll new file mode 100644 index 00000000000..6f54fbbe3e6 --- /dev/null +++ b/test/Analysis/GlobalsModRef/nonescaping-noalias.ll @@ -0,0 +1,62 @@ +; RUN: opt < %s -globalsmodref-aa -gvn -S | FileCheck %s +; +; This tests the safe no-alias conclusions of GMR -- when there is +; a non-escaping global as one indentified underlying object and some pointer +; that would inherently have escaped any other function as the other underlying +; pointer of an alias query. + +@g1 = internal global i32 0 + +define i32 @test1(i32* %param) { +; Ensure that we can fold a store to a load of a global across a store to +; a parameter when the global is non-escaping. +; +; CHECK-LABEL: @test1( +; CHECK: store i32 42, i32* @g1 +; CHECK-NOT: load i32 +; CHECK: ret i32 42 +entry: + store i32 42, i32* @g1 + store i32 7, i32* %param + %v = load i32, i32* @g1 + ret i32 %v +} + +declare i32* @f() + +define i32 @test2() { +; Ensure that we can fold a store to a load of a global across a store to +; the pointer returned by a function call. Since the global could not escape, +; this function cannot be returning its address. +; +; CHECK-LABEL: @test2( +; CHECK: store i32 42, i32* @g1 +; CHECK-NOT: load i32 +; CHECK: ret i32 42 +entry: + %ptr = call i32* @f() readnone + store i32 42, i32* @g1 + store i32 7, i32* %ptr + %v = load i32, i32* @g1 + ret i32 %v +} + +@g2 = external global i32* + +define i32 @test3() { +; Ensure that we can fold a store to a load of a global across a store to +; the pointer loaded from that global. Because the global does not escape, it +; cannot alias a pointer loaded out of a global. +; +; CHECK-LABEL: @test3( +; CHECK: store i32 42, i32* @g1 +; CHECK: store i32 7, i32* +; CHECK-NOT: load i32 +; CHECK: ret i32 42 +entry: + store i32 42, i32* @g1 + %ptr1 = load i32*, i32** @g2 + store i32 7, i32* %ptr1 + %v = load i32, i32* @g1 + ret i32 %v +} -- 2.34.1