[llvm] r193251 - Use address-taken to disambiguate global variable and indirect memops.

Chandler Carruth chandlerc at google.com
Sat Oct 26 17:40:52 PDT 2013


On Oct 26, 2013 5:37 PM, "Chris Lattner" <clattner at apple.com> wrote:
>
>
> On Oct 23, 2013, at 10:28 AM, Shuxin Yang <shuxin.llvm at gmail.com> wrote:
>
> > Author: shuxin_yang
> > Date: Wed Oct 23 12:28:19 2013
> > New Revision: 193251
> >
> > URL: http://llvm.org/viewvc/llvm-project?rev=193251&view=rev
> > Log:
> > Use address-taken to disambiguate global variable and indirect memops.
> >
> > Major steps include:
> > 1). introduces a not-addr-taken bit-field in GlobalVariable
> > 2). GlobalOpt pass sets "not-address-taken" if it proves a global
varirable
> >    dosen't have its address taken.
> > 3). AA use this info for disambiguation.
>
> Shuxin please revert this immediately.
>
> This is a major IR change.  Please propose IR changes (e.g. new bits in
key datatypes) on llvmdev before making any changes like this.  This is
completely inappropriate for post-commit review.

He actually asked for precommit review and had a response on that thread on
Tuesday. The response wasn't an LGTM tho and in no way did it look like it
was done being reviewed.

>
> -Chris
>
>
> >
> > Added:
> >    llvm/trunk/test/Analysis/BasicAA/noaddrtaken.ll
> > Modified:
> >    llvm/trunk/docs/LangRef.rst
> >    llvm/trunk/include/llvm/IR/GlobalVariable.h
> >    llvm/trunk/lib/Analysis/BasicAliasAnalysis.cpp
> >    llvm/trunk/lib/AsmParser/LLLexer.cpp
> >    llvm/trunk/lib/AsmParser/LLParser.cpp
> >    llvm/trunk/lib/AsmParser/LLToken.h
> >    llvm/trunk/lib/Bitcode/Reader/BitcodeReader.cpp
> >    llvm/trunk/lib/Bitcode/Writer/BitcodeWriter.cpp
> >    llvm/trunk/lib/IR/AsmWriter.cpp
> >    llvm/trunk/lib/IR/Globals.cpp
> >    llvm/trunk/lib/Transforms/IPO/GlobalOpt.cpp
> >    llvm/trunk/test/Transforms/GlobalOpt/2009-03-07-PromotePtrToBool.ll
> >
 llvm/trunk/test/Transforms/GlobalOpt/2009-11-16-MallocSingleStoreToGlobalVar.ll
> >    llvm/trunk/test/Transforms/GlobalOpt/atomic.ll
> >    llvm/trunk/test/Transforms/GlobalOpt/globalsra-unknown-index.ll
> >    llvm/trunk/test/Transforms/GlobalOpt/integer-bool.ll
> >    llvm/trunk/test/Transforms/GlobalOpt/unnamed-addr.ll
> >
> > Modified: llvm/trunk/docs/LangRef.rst
> > URL:
http://llvm.org/viewvc/llvm-project/llvm/trunk/docs/LangRef.rst?rev=193251&r1=193250&r2=193251&view=diff
> >
==============================================================================
> > --- llvm/trunk/docs/LangRef.rst (original)
> > +++ llvm/trunk/docs/LangRef.rst Wed Oct 23 12:28:19 2013
> > @@ -511,6 +511,9 @@ module, including those with external li
> > ``@llvm.used``. This assumption may be suppressed by marking the
> > variable with ``externally_initialized``.
> >
> > +If a global variable dose not have its address taken, it will be
optionally
> > +flagged ``notaddrtaken``.
> > +
> > An explicit alignment may be specified for a global, which must be a
> > power of 2. If not present, or if the alignment is set to zero, the
> > alignment of the global is set by the target to whatever it feels
> >
> > Modified: llvm/trunk/include/llvm/IR/GlobalVariable.h
> > URL:
http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/IR/GlobalVariable.h?rev=193251&r1=193250&r2=193251&view=diff
> >
==============================================================================
> > --- llvm/trunk/include/llvm/IR/GlobalVariable.h (original)
> > +++ llvm/trunk/include/llvm/IR/GlobalVariable.h Wed Oct 23 12:28:19 2013
> > @@ -48,6 +48,7 @@ class GlobalVariable : public GlobalValu
> >                                                // can change from its
initial
> >                                                // value before global
> >                                                // initializers are run?
> > +  bool notAddrTaken : 1;                       // Dose not have
address taken.
> >
> > public:
> >   // allocate space for exactly one operand
> > @@ -174,6 +175,9 @@ public:
> >     isExternallyInitializedConstant = Val;
> >   }
> >
> > +  void setAddressMaybeTaken(bool Val) { notAddrTaken = !Val; }
> > +  bool AddressMaybeTaken(void) const { return !notAddrTaken; }
> > +
> >   /// copyAttributesFrom - copy all additional attributes (those not
needed to
> >   /// create a GlobalVariable) from the GlobalVariable Src to this one.
> >   void copyAttributesFrom(const GlobalValue *Src);
> >
> > Modified: llvm/trunk/lib/Analysis/BasicAliasAnalysis.cpp
> > URL:
http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Analysis/BasicAliasAnalysis.cpp?rev=193251&r1=193250&r2=193251&view=diff
> >
==============================================================================
> > --- llvm/trunk/lib/Analysis/BasicAliasAnalysis.cpp (original)
> > +++ llvm/trunk/lib/Analysis/BasicAliasAnalysis.cpp Wed Oct 23 12:28:19
2013
> > @@ -1238,6 +1238,17 @@ BasicAliasAnalysis::aliasCheck(const Val
> >       return NoAlias;
> >     if (isEscapeSource(O2) && isNonEscapingLocalObject(O1))
> >       return NoAlias;
> > +
> > +    // If one object is a global variable without address taken, the
other one
> > +    // is a different object, they will not alias because the global
variable
> > +    // in question cannot be indirectly accessed.
> > +    if (const GlobalVariable *GV = dyn_cast<GlobalVariable>(O1))
> > +      if (!GV->AddressMaybeTaken())
> > +        return NoAlias;
> > +
> > +    if (const GlobalVariable *GV = dyn_cast<GlobalVariable>(O2))
> > +      if (!GV->AddressMaybeTaken())
> > +        return NoAlias;
> >   }
> >
> >   // If the size of one access is larger than the entire object on the
other
> >
> > Modified: llvm/trunk/lib/AsmParser/LLLexer.cpp
> > URL:
http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/AsmParser/LLLexer.cpp?rev=193251&r1=193250&r2=193251&view=diff
> >
==============================================================================
> > --- llvm/trunk/lib/AsmParser/LLLexer.cpp (original)
> > +++ llvm/trunk/lib/AsmParser/LLLexer.cpp Wed Oct 23 12:28:19 2013
> > @@ -504,6 +504,7 @@ lltok::Kind LLLexer::LexIdentifier() {
> >   KEYWORD(zeroinitializer);
> >   KEYWORD(undef);
> >   KEYWORD(null);
> > +  KEYWORD(notaddrtaken);
> >   KEYWORD(to);
> >   KEYWORD(tail);
> >   KEYWORD(target);
> >
> > Modified: llvm/trunk/lib/AsmParser/LLParser.cpp
> > URL:
http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/AsmParser/LLParser.cpp?rev=193251&r1=193250&r2=193251&view=diff
> >
==============================================================================
> > --- llvm/trunk/lib/AsmParser/LLParser.cpp (original)
> > +++ llvm/trunk/lib/AsmParser/LLParser.cpp Wed Oct 23 12:28:19 2013
> > @@ -704,7 +704,7 @@ bool LLParser::ParseGlobal(const std::st
> >                            unsigned Linkage, bool HasLinkage,
> >                            unsigned Visibility) {
> >   unsigned AddrSpace;
> > -  bool IsConstant, UnnamedAddr, IsExternallyInitialized;
> > +  bool IsConstant, UnnamedAddr, IsExternallyInitialized, notAddrTaken;
> >   GlobalVariable::ThreadLocalMode TLM;
> >   LocTy UnnamedAddrLoc;
> >   LocTy IsExternallyInitializedLoc;
> > @@ -719,6 +719,7 @@ bool LLParser::ParseGlobal(const std::st
> >                          IsExternallyInitialized,
> >                          &IsExternallyInitializedLoc) ||
> >       ParseGlobalType(IsConstant) ||
> > +      ParseOptionalToken(lltok::kw_notaddrtaken, notAddrTaken) ||
> >       ParseType(Ty, TyLoc))
> >     return true;
> >
> > @@ -776,6 +777,7 @@ bool LLParser::ParseGlobal(const std::st
> >   GV->setLinkage((GlobalValue::LinkageTypes)Linkage);
> >   GV->setVisibility((GlobalValue::VisibilityTypes)Visibility);
> >   GV->setExternallyInitialized(IsExternallyInitialized);
> > +  GV->setAddressMaybeTaken(!notAddrTaken);
> >   GV->setThreadLocalMode(TLM);
> >   GV->setUnnamedAddr(UnnamedAddr);
> >
> >
> > Modified: llvm/trunk/lib/AsmParser/LLToken.h
> > URL:
http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/AsmParser/LLToken.h?rev=193251&r1=193250&r2=193251&view=diff
> >
==============================================================================
> > --- llvm/trunk/lib/AsmParser/LLToken.h (original)
> > +++ llvm/trunk/lib/AsmParser/LLToken.h Wed Oct 23 12:28:19 2013
> > @@ -51,6 +51,7 @@ namespace lltok {
> >     kw_localdynamic, kw_initialexec, kw_localexec,
> >     kw_zeroinitializer,
> >     kw_undef, kw_null,
> > +    kw_notaddrtaken,
> >     kw_to,
> >     kw_tail,
> >     kw_target,
> >
> > Modified: llvm/trunk/lib/Bitcode/Reader/BitcodeReader.cpp
> > URL:
http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Bitcode/Reader/BitcodeReader.cpp?rev=193251&r1=193250&r2=193251&view=diff
> >
==============================================================================
> > --- llvm/trunk/lib/Bitcode/Reader/BitcodeReader.cpp (original)
> > +++ llvm/trunk/lib/Bitcode/Reader/BitcodeReader.cpp Wed Oct 23 12:28:19
2013
> > @@ -1848,6 +1848,9 @@ bool BitcodeReader::ParseModule(bool Res
> >         new GlobalVariable(*TheModule, Ty, isConstant, Linkage, 0, "",
0,
> >                            TLM, AddressSpace, ExternallyInitialized);
> >       NewGV->setAlignment(Alignment);
> > +      if (Record.size() > 10)
> > +        NewGV->setAddressMaybeTaken(Record[10]);
> > +
> >       if (!Section.empty())
> >         NewGV->setSection(Section);
> >       NewGV->setVisibility(Visibility);
> >
> > Modified: llvm/trunk/lib/Bitcode/Writer/BitcodeWriter.cpp
> > URL:
http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Bitcode/Writer/BitcodeWriter.cpp?rev=193251&r1=193250&r2=193251&view=diff
> >
==============================================================================
> > --- llvm/trunk/lib/Bitcode/Writer/BitcodeWriter.cpp (original)
> > +++ llvm/trunk/lib/Bitcode/Writer/BitcodeWriter.cpp Wed Oct 23 12:28:19
2013
> > @@ -616,11 +616,13 @@ static void WriteModuleInfo(const Module
> >     Vals.push_back(GV->hasSection() ? SectionMap[GV->getSection()] : 0);
> >     if (GV->isThreadLocal() ||
> >         GV->getVisibility() != GlobalValue::DefaultVisibility ||
> > -        GV->hasUnnamedAddr() || GV->isExternallyInitialized()) {
> > +        GV->hasUnnamedAddr() || GV->isExternallyInitialized() ||
> > +        !GV->AddressMaybeTaken()) {
> >       Vals.push_back(getEncodedVisibility(GV));
> >       Vals.push_back(getEncodedThreadLocalMode(GV));
> >       Vals.push_back(GV->hasUnnamedAddr());
> >       Vals.push_back(GV->isExternallyInitialized());
> > +      Vals.push_back(GV->AddressMaybeTaken());
> >     } else {
> >       AbbrevToUse = SimpleGVarAbbrev;
> >     }
> >
> > Modified: llvm/trunk/lib/IR/AsmWriter.cpp
> > URL:
http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/IR/AsmWriter.cpp?rev=193251&r1=193250&r2=193251&view=diff
> >
==============================================================================
> > --- llvm/trunk/lib/IR/AsmWriter.cpp (original)
> > +++ llvm/trunk/lib/IR/AsmWriter.cpp Wed Oct 23 12:28:19 2013
> > @@ -1459,6 +1459,7 @@ void AssemblyWriter::printGlobal(const G
> >   if (GV->hasUnnamedAddr()) Out << "unnamed_addr ";
> >   if (GV->isExternallyInitialized()) Out << "externally_initialized ";
> >   Out << (GV->isConstant() ? "constant " : "global ");
> > +  if (!GV->AddressMaybeTaken()) Out << "notaddrtaken ";
> >   TypePrinter.print(GV->getType()->getElementType(), Out);
> >
> >   if (GV->hasInitializer()) {
> >
> > Modified: llvm/trunk/lib/IR/Globals.cpp
> > URL:
http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/IR/Globals.cpp?rev=193251&r1=193250&r2=193251&view=diff
> >
==============================================================================
> > --- llvm/trunk/lib/IR/Globals.cpp (original)
> > +++ llvm/trunk/lib/IR/Globals.cpp Wed Oct 23 12:28:19 2013
> > @@ -99,6 +99,7 @@ GlobalVariable::GlobalVariable(Type *Ty,
> >   }
> >
> >   LeakDetector::addGarbageObject(this);
> > +  setAddressMaybeTaken(true);
> > }
> >
> > GlobalVariable::GlobalVariable(Module &M, Type *Ty, bool constant,
> > @@ -125,6 +126,7 @@ GlobalVariable::GlobalVariable(Module &M
> >     Before->getParent()->getGlobalList().insert(Before, this);
> >   else
> >     M.getGlobalList().push_back(this);
> > +  setAddressMaybeTaken(true);
> > }
> >
> > void GlobalVariable::setParent(Module *parent) {
> > @@ -185,6 +187,7 @@ void GlobalVariable::copyAttributesFrom(
> >   GlobalValue::copyAttributesFrom(Src);
> >   const GlobalVariable *SrcVar = cast<GlobalVariable>(Src);
> >   setThreadLocal(SrcVar->isThreadLocal());
> > +  setAddressMaybeTaken(SrcVar->AddressMaybeTaken());
> > }
> >
> >
> >
> > Modified: llvm/trunk/lib/Transforms/IPO/GlobalOpt.cpp
> > URL:
http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Transforms/IPO/GlobalOpt.cpp?rev=193251&r1=193250&r2=193251&view=diff
> >
==============================================================================
> > --- llvm/trunk/lib/Transforms/IPO/GlobalOpt.cpp (original)
> > +++ llvm/trunk/lib/Transforms/IPO/GlobalOpt.cpp Wed Oct 23 12:28:19 2013
> > @@ -1723,6 +1723,7 @@ bool GlobalOpt::ProcessGlobal(GlobalVari
> >   if (GlobalStatus::analyzeGlobal(GV, GS))
> >     return false;
> >
> > +  GV->setAddressMaybeTaken(false);
> >   if (!GS.IsCompared && !GV->hasUnnamedAddr()) {
> >     GV->setUnnamedAddr(true);
> >     NumUnnamed++;
> >
> > Added: llvm/trunk/test/Analysis/BasicAA/noaddrtaken.ll
> > URL:
http://llvm.org/viewvc/llvm-project/llvm/trunk/test/Analysis/BasicAA/noaddrtaken.ll?rev=193251&view=auto
> >
==============================================================================
> > --- llvm/trunk/test/Analysis/BasicAA/noaddrtaken.ll (added)
> > +++ llvm/trunk/test/Analysis/BasicAA/noaddrtaken.ll Wed Oct 23 12:28:19
2013
> > @@ -0,0 +1,29 @@
> > +; RUN: opt < %s -basicaa -aa-eval -print-all-alias-modref-info 2>&1 |
FileCheck %s
> > +
> > +; ModuleID = 'b.c'
> > +target datalayout =
"e-p:64:64:64-i1:8:8-i8:8:8-i16:16:16-i32:32:32-i64:64:64-f32:32:32-f64:64:64-v64:64:64-v128:128:128-a0:0:64-s0:64:64-f80:128:128-n8:16:32:64-S128"
> > +target triple = "x86_64-apple-macosx10.8.0"
> > +
> > +; CHECK: NoAlias:   i32* %p, i32* @xyz
> > +
> > +;@xyz = global i32 12, align 4
> > + at xyz = internal unnamed_addr global notaddrtaken i32 12, align 4
> > +
> > +; Function Attrs: nounwind ssp uwtable
> > +define i32 @foo(i32* nocapture %p, i32* nocapture %q) #0 {
> > +entry:
> > +  %0 = load i32* @xyz, align 4, !tbaa !0
> > +  %inc = add nsw i32 %0, 1
> > +  store i32 %inc, i32* @xyz, align 4, !tbaa !0
> > +  store i32 1, i32* %p, align 4, !tbaa !0
> > +  %1 = load i32* @xyz, align 4, !tbaa !0
> > +  store i32 %1, i32* %q, align 4, !tbaa !0
> > +  ret i32 undef
> > +}
> > +
> > +attributes #0 = { nounwind ssp uwtable "less-precise-fpmad"="false"
"no-frame-pointer-elim"="true" "no-frame-pointer-elim-non-leaf"
"no-infs-fp-math"="false" "no-nans-fp-math"="false"
"stack-protector-buffer-size"="8" "unsafe-fp-math"="false"
"use-soft-float"="false" }
> > +
> > +!0 = metadata !{metadata !1, metadata !1, i64 0}
> > +!1 = metadata !{metadata !"int", metadata !2, i64 0}
> > +!2 = metadata !{metadata !"omnipotent char", metadata !3, i64 0}
> > +!3 = metadata !{metadata !"Simple C/C++ TBAA"}
> >
> > Modified:
llvm/trunk/test/Transforms/GlobalOpt/2009-03-07-PromotePtrToBool.ll
> > URL:
http://llvm.org/viewvc/llvm-project/llvm/trunk/test/Transforms/GlobalOpt/2009-03-07-PromotePtrToBool.ll?rev=193251&r1=193250&r2=193251&view=diff
> >
==============================================================================
> > --- llvm/trunk/test/Transforms/GlobalOpt/2009-03-07-PromotePtrToBool.ll
(original)
> > +++ llvm/trunk/test/Transforms/GlobalOpt/2009-03-07-PromotePtrToBool.ll
Wed Oct 23 12:28:19 2013
> > @@ -1,4 +1,4 @@
> > -; RUN: opt < %s -globalopt -S | grep "@X = internal unnamed_addr
global i32"
> > +; RUN: opt < %s -globalopt -S | grep "@X = internal unnamed_addr
global notaddrtaken i32"
> > target datalayout =
"e-p:32:32:32-i1:8:8-i8:8:8-i16:16:16-i32:32:32-i64:32:64-f32:32:32-f64:32:64-v64:64:64-v128:128:128-a0:0:64-f80:128:128"
> > target triple = "i386-apple-darwin7"
> > @X = internal global i32* null                ; <i32**> [#uses=2]
> >
> > Modified:
llvm/trunk/test/Transforms/GlobalOpt/2009-11-16-MallocSingleStoreToGlobalVar.ll
> > URL:
http://llvm.org/viewvc/llvm-project/llvm/trunk/test/Transforms/GlobalOpt/2009-11-16-MallocSingleStoreToGlobalVar.ll?rev=193251&r1=193250&r2=193251&view=diff
> >
==============================================================================
> > ---
llvm/trunk/test/Transforms/GlobalOpt/2009-11-16-MallocSingleStoreToGlobalVar.ll
(original)
> > +++
llvm/trunk/test/Transforms/GlobalOpt/2009-11-16-MallocSingleStoreToGlobalVar.ll
Wed Oct 23 12:28:19 2013
> > @@ -8,7 +8,7 @@ target datalayout = "e-p:64:64:64-i1:8:8
> > target triple = "x86_64-apple-darwin10.0"
> >
> > @TOP = internal global i64* null                    ; <i64**> [#uses=2]
> > -; CHECK: @TOP = internal unnamed_addr global i64* null
> > +; CHECK: @TOP = internal unnamed_addr global notaddrtaken i64* null
> > @channelColumns = internal global i64 0             ; <i64*> [#uses=2]
> >
> > ; Derived from @DescribeChannel() in yacr2
> >
> > Modified: llvm/trunk/test/Transforms/GlobalOpt/atomic.ll
> > URL:
http://llvm.org/viewvc/llvm-project/llvm/trunk/test/Transforms/GlobalOpt/atomic.ll?rev=193251&r1=193250&r2=193251&view=diff
> >
==============================================================================
> > --- llvm/trunk/test/Transforms/GlobalOpt/atomic.ll (original)
> > +++ llvm/trunk/test/Transforms/GlobalOpt/atomic.ll Wed Oct 23 12:28:19
2013
> > @@ -3,8 +3,8 @@
> > @GV1 = internal global i64 1
> > @GV2 = internal global i32 0
> >
> > -; CHECK: @GV1 = internal unnamed_addr constant i64 1
> > -; CHECK: @GV2 = internal unnamed_addr global i32 0
> > +; CHECK: @GV1 = internal unnamed_addr constant notaddrtaken i64 1
> > +; CHECK: @GV2 = internal unnamed_addr global notaddrtaken i32 0
> >
> > define void @test1() {
> > entry:
> >
> > Modified:
llvm/trunk/test/Transforms/GlobalOpt/globalsra-unknown-index.ll
> > URL:
http://llvm.org/viewvc/llvm-project/llvm/trunk/test/Transforms/GlobalOpt/globalsra-unknown-index.ll?rev=193251&r1=193250&r2=193251&view=diff
> >
==============================================================================
> > --- llvm/trunk/test/Transforms/GlobalOpt/globalsra-unknown-index.ll
(original)
> > +++ llvm/trunk/test/Transforms/GlobalOpt/globalsra-unknown-index.ll Wed
Oct 23 12:28:19 2013
> > @@ -1,5 +1,5 @@
> > ; RUN: opt < %s -globalopt -S > %t
> > -; RUN: grep "@Y = internal unnamed_addr global \[3 x [%]struct.X\]
zeroinitializer" %t
> > +; RUN: grep "@Y = internal unnamed_addr global notaddrtaken \[3 x
[%]struct.X\] zeroinitializer" %t
> > ; RUN: grep load %t | count 6
> > ; RUN: grep "add i32 [%]a, [%]b" %t | count 3
> >
> >
> > Modified: llvm/trunk/test/Transforms/GlobalOpt/integer-bool.ll
> > URL:
http://llvm.org/viewvc/llvm-project/llvm/trunk/test/Transforms/GlobalOpt/integer-bool.ll?rev=193251&r1=193250&r2=193251&view=diff
> >
==============================================================================
> > --- llvm/trunk/test/Transforms/GlobalOpt/integer-bool.ll (original)
> > +++ llvm/trunk/test/Transforms/GlobalOpt/integer-bool.ll Wed Oct 23
12:28:19 2013
> > @@ -4,7 +4,7 @@
> > @G = internal addrspace(1) global i32 0
> > ; CHECK: @G
> > ; CHECK: addrspace(1)
> > -; CHECK: global i1 false
> > +; CHECK: global notaddrtaken i1 false
> >
> > define void @set1() {
> >   store i32 0, i32 addrspace(1)* @G
> >
> > Modified: llvm/trunk/test/Transforms/GlobalOpt/unnamed-addr.ll
> > URL:
http://llvm.org/viewvc/llvm-project/llvm/trunk/test/Transforms/GlobalOpt/unnamed-addr.ll?rev=193251&r1=193250&r2=193251&view=diff
> >
==============================================================================
> > --- llvm/trunk/test/Transforms/GlobalOpt/unnamed-addr.ll (original)
> > +++ llvm/trunk/test/Transforms/GlobalOpt/unnamed-addr.ll Wed Oct 23
12:28:19 2013
> > @@ -6,10 +6,10 @@
> > @d = internal constant [4 x i8] c"foo\00", align 1
> > @e = linkonce_odr global i32 0
> >
> > -; CHECK: @a = internal global i32 0, align 4
> > +; CHECK: @a = internal global notaddrtaken i32 0, align 4
> > ; CHECK: @b = internal global i32 0, align 4
> > -; CHECK: @c = internal unnamed_addr global i32 0, align 4
> > -; CHECK: @d = internal unnamed_addr constant [4 x i8] c"foo\00", align
1
> > +; CHECK: @c = internal unnamed_addr global notaddrtaken i32 0, align 4
> > +; CHECK: @d = internal unnamed_addr constant notaddrtaken [4 x i8]
c"foo\00", align 1
> > ; CHECK: @e = linkonce_odr global i32 0
> >
> > define i32 @get_e() {
> >
> >
> > _______________________________________________
> > llvm-commits mailing list
> > llvm-commits at cs.uiuc.edu
> > http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20131026/c6ba56c1/attachment.html>


More information about the llvm-commits mailing list