[llvm-commits] [llvm] r164480 - /llvm/trunk/lib/Transforms/IPO/PassManagerBuilder.cpp

Chandler Carruth chandlerc at gmail.com
Mon Sep 24 19:44:38 PDT 2012


David, I think r164585 should fix this. I've not gotten a reduced test
case, and I'm not 100% confident I was running your reproduction correctly,
so please let me know if you're still seeing the crash.

Thanks for the report! Keep any others coming!


On Mon, Sep 24, 2012 at 9:24 AM, David Dean <david_dean at apple.com> wrote:

> Fallout as requested:
>
> A crasher. Preprocessed source and shell script attached.
>
> llvm[6]: Compiling SubEngine.cpp for Release build
> Stack dump:
> 0.      Program arguments:
> /Users/buildslave/zorg/buildbot/smooshlab/slave-0.8/build.apple-clang-x86_64-darwin10-R/clang.obj/clang-9999.99_clang.src.roots/clang-9999.99_clang.src~obj/stage1-install-x86_64/bin/clang
> -cc1 -triple i386-apple-macosx10.6.0 -emit-obj -disable-free
> -disable-llvm-verifier -main-file-name Store.cpp -pic-level 2
> -mdisable-fp-elim -relaxed-aliasing -masm-verbose -target-cpu yonah
> -target-linker-version 123.2 -g -coverage-file
> /Users/buildslave/zorg/buildbot/smooshlab/slave-0.8/build.apple-clang-x86_64-darwin10-R/clang.obj/clang-9999.99_clang.src.roots/clang-9999.99_clang.src~obj/i386/tools/clang/lib/StaticAnalyzer/Core/Release/Store.o
> -resource-dir
> /Users/buildslave/zorg/buildbot/smooshlab/slave-0.8/build.apple-clang-x86_64-darwin10-R/clang.obj/clang-9999.99_clang.src.roots/clang-9999.99_clang.src~obj/stage1-install-x86_64/bin/../lib/clang/4.2
> -dependency-file
> /Users/buildslave/zorg/buildbot/smooshlab/slave-0.8/build.apple-clang-x86_64-darwin10-R/clang.obj/clang-9999.99_clang.src.roots/clang-9999.99_clang.src~obj/i386/tools/clang/lib/StaticAnalyzer/Core/Release/Store.d.tmp
> -MP -MT
> /Users/buildslave/zorg/buildbot/smooshlab/slave-0.8/build.apple-clang-x86_64-darwin10-R/clang.obj/clang-9999.99_clang.src.roots/clang-9999.99_clang.src~obj/i386/tools/clang/lib/StaticAnalyzer/Core/Release/Store.o
> -MT
> /Users/buildslave/zorg/buildbot/smooshlab/slave-0.8/build.apple-clang-x86_64-darwin10-R/clang.obj/clang-9999.99_clang.src.roots/clang-9999.99_clang.src~obj/i386/tools/clang/lib/StaticAnalyzer/Core/Release/Store.d
> -D NDEBUG -D _GNU_SOURCE -D __STDC_CONSTANT_MACROS -D __STDC_FORMAT_MACROS
> -D __STDC_LIMIT_MACROS -D CLANG_VENDOR="Apple " -D
> CLANG_REPOSITORY_STRING="tags/Apple/clang-9999.99" -D
> DISABLE_DEFAULT_STRICT_ALIASING -I
> /Users/buildslave/zorg/buildbot/smooshlab/slave-0.8/build.apple-clang-x86_64-darwin10-R/clang.obj/clang-9999.99_clang.src.roots/clang-9999.99_clang.src~obj/i386/include
> -I
> /Users/buildslave/zorg/buildbot/smooshlab/slave-0.8/build.apple-clang-x86_64-darwin10-R/clang.obj/clang-9999.99_clang.src.roots/clang-9999.99_clang.src~obj/i386/tools/clang/lib/StaticAnalyzer/Core
> -I
> /Users/buildslave/zorg/buildbot/smooshlab/slave-0.8/build.apple-clang-x86_64-darwin10-R/clang.src/src/include
> -I
> /Users/buildslave/zorg/buildbot/smooshlab/slave-0.8/build.apple-clang-x86_64-darwin10-R/clang.src/src/tools/clang/lib/StaticAnalyzer/Core
> -I
> /Users/buildslave/zorg/buildbot/smooshlab/slave-0.8/build.apple-clang-x86_64-darwin10-R/clang.src/src/tools/clang/lib/StaticAnalyzer/Core/../../../include
> -I
> /Users/buildslave/zorg/buildbot/smooshlab/slave-0.8/build.apple-clang-x86_64-darwin10-R/clang.obj/clang-9999.99_clang.src.roots/clang-9999.99_clang.src~obj/i386/tools/clang/lib/StaticAnalyzer/Core/../../../include
> -fmodule-cache-path /var/tmp/clang-module-cache -O2 -Woverloaded-virtual
> -Wcast-qual -Wno-long-long -Wall -W -Wno-unused-parameter -Wwrite-strings
> -Wcovered-switch-default -pedantic -fconst-strings -fdeprecated-macro
> -fdebug-compilation-dir
> /Users/buildslave/zorg/buildbot/smooshlab/slave-0.8/build.apple-clang-x86_64-darwin10-R/clang.obj/clang-9999.99_clang.src.roots/clang-9999.99_clang.src~obj/i386/tools/clang/lib/StaticAnalyzer/Core
> -ferror-limit 19 -fmessage-length 0 -fvisibility-inlines-hidden
> -mstackrealign -fblocks -fno-rtti -fobjc-runtime=macosx-fragile-10.6.0
> -fobjc-default-synthesize-properties -fno-common -fdiagnostics-show-option
> -o
> /Users/buildslave/zorg/buildbot/smooshlab/slave-0.8/build.apple-clang-x86_64-darwin10-R/clang.obj/clang-9999.99_clang.src.roots/clang-9999.99_clang.src~obj/i386/tools/clang/lib/StaticAnalyzer/Core/Release/Store.o
> -x c++
> /Users/buildslave/zorg/buildbot/smooshlab/slave-0.8/build.apple-clang-x86_64-darwin10-R/clang.src/src/tools/clang/lib/StaticAnalyzer/Core/Store.cpp
> -dwarf-debug-flags
> /Users/buildslave/zorg/buildbot/smooshlab/slave-0.8/build.apple-clang-x86_64-darwin10-R/clang.obj/clang-9999.99_clang.src.roots/clang-9999.99_clang.src~obj/stage1-install-x86_64/bin/clang
> -arch i386 -I
> /Users/buildslave/zorg/buildbot/smooshlab/slave-0.8/build.apple-clang-x86_64-darwin10-R/clang.obj/clang-9999.99_clang.src.roots/clang-9999.99_clang.src~obj/i386/include
> -I
> /Users/buildslave/zorg/buildbot/smooshlab/slave-0.8/build.apple-clang-x86_64-darwin10-R/clang.obj/clang-9999.99_clang.src.roots/clang-9999.99_clang.src~obj/i386/tools/clang/lib/StaticAnalyzer/Core
> -I
> /Users/buildslave/zorg/buildbot/smooshlab/slave-0.8/build.apple-clang-x86_64-darwin10-R/clang.src/src/include
> -I
> /Users/buildslave/zorg/buildbot/smooshlab/slave-0.8/build.apple-clang-x86_64-darwin10-R/clang.src/src/tools/clang/lib/StaticAnalyzer/Core
> -D NDEBUG -D _GNU_SOURCE -D __STDC_CONSTANT_MACROS -D __STDC_FORMAT_MACROS
> -D __STDC_LIMIT_MACROS -I
> /Users/buildslave/zorg/buildbot/smooshlab/slave-0.8/build.apple-clang-x86_64-darwin10-R/clang.src/src/tools/clang/lib/StaticAnalyzer/Core/../../../include
> -I
> /Users/buildslave/zorg/buildbot/smooshlab/slave-0.8/build.apple-clang-x86_64-darwin10-R/clang.obj/clang-9999.99_clang.src.roots/clang-9999.99_clang.src~obj/i386/tools/clang/lib/StaticAnalyzer/Core/../../../include
> -D CLANG_VENDOR="Apple " -D
> CLANG_REPOSITORY_STRING="tags/Apple/clang-9999.99" -O2 -fno-stack-protector
> -fvisibility-inlines-hidden -fno-exceptions -fno-rtti -fno-common
> -Woverloaded-virtual -Wcast-qual -fno-strict-aliasing -m32 -pedantic
> -Wno-long-long -Wall -W -Wno-unused-parameter -Wwrite-strings -D
> DISABLE_DEFAULT_STRICT_ALIASING -g -Wcovered-switch-default -c -MMD -MP -MF
> /Users/buildslave/zorg/buildbot/smooshlab/slave-0.8/build.apple-clang-x86_64-darwin10-R/clang.obj/clang-9999.99_clang.src.roots/clang-9999.99_clang.src~obj/i386/tools/clang/lib/StaticAnalyzer/Core/Release/Store.d.tmp
> -MT
> /Users/buildslave/zorg/buildbot/smooshlab/slave-0.8/build.apple-clang-x86_64-darwin10-R/clang.obj/clang-9999.99_clang.src.roots/clang-9999.99_clang.src~obj/i386/tools/clang/lib/StaticAnalyzer/Core/Release/Store.o
> -MT
> /Users/buildslave/zorg/buildbot/smooshlab/slave-0.8/build.apple-clang-x86_64-darwin10-R/clang.obj/clang-9999.99_clang.src.roots/clang-9999.99_clang.src~obj/i386/tools/clang/lib/StaticAnalyzer/Core/Release/Store.d
> /Users/buildslave/zorg/buildbot/smooshlab/slave-0.8/build.apple-clang-x86_64-darwin10-R/clang.src/src/tools/clang/lib/StaticAnalyzer/Core/Store.cpp
> -o
> /Users/buildslave/zorg/buildbot/smooshlab/slave-0.8/build.apple-clang-x86_64-darwin10-R/clang.obj/clang-9999.99_clang.src.roots/clang-9999.99_clang.src~obj/i386/tools/clang/lib/StaticAnalyzer/Core/Release/Store.o
> -mlinker-version=123.2 -mtune=core2 -mmacosx-version-min=10.6
> 1.      <eof> parser at end of file
> 2.      Per-module optimization passes
> 3.      Running pass 'CallGraph Pass Manager' on module
> '/Users/buildslave/zorg/buildbot/smooshlab/slave-0.8/build.apple-clang-x86_64-darwin10-R/clang.src/src/tools/clang/lib/StaticAnalyzer/Core/Store.cpp'.
> 4.      Running pass 'SROA' on function
> '@_ZN5clang4ento12StoreManager13getLValueIvarEPKNS_12ObjCIvarDeclENS0_4SValE'
> clang: error: unable to execute command: Segmentation fault
> clang: error: clang frontend command failed due to signal (use -v to see
> invocation)
>
>
>
> On 23 Sep 2012, at 6:10 PM, Chandler Carruth <chandlerc at gmail.com> wrote:
>
> > Author: chandlerc
> > Date: Sun Sep 23 20:10:25 2012
> > New Revision: 164480
> >
> > URL: http://llvm.org/viewvc/llvm-project?rev=164480&view=rev
> > Log:
> > Enable the new SROA pass by default.
> >
> > Queue the fallout. ;]
> >
> > Modified:
> >    llvm/trunk/lib/Transforms/IPO/PassManagerBuilder.cpp
> >
> > Modified: llvm/trunk/lib/Transforms/IPO/PassManagerBuilder.cpp
> > URL:
> http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Transforms/IPO/PassManagerBuilder.cpp?rev=164480&r1=164479&r2=164480&view=diff
> >
> ==============================================================================
> > --- llvm/trunk/lib/Transforms/IPO/PassManagerBuilder.cpp (original)
> > +++ llvm/trunk/lib/Transforms/IPO/PassManagerBuilder.cpp Sun Sep 23
> 20:10:25 2012
> > @@ -41,7 +41,7 @@
> >   cl::desc("Run GVN instead of Early CSE after vectorization passes"));
> >
> > static cl::opt<bool> UseNewSROA("use-new-sroa",
> > -  cl::init(false), cl::Hidden,
> > +  cl::init(true), cl::Hidden,
> >   cl::desc("Enable the new, experimental SROA pass"));
> >
> > PassManagerBuilder::PassManagerBuilder() {
> >
> >
> > _______________________________________________
> > llvm-commits mailing list
> > llvm-commits at cs.uiuc.edu
> > http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>
> -David
>
>
>
> _______________________________________________
> 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/20120924/2604158e/attachment.html>


More information about the llvm-commits mailing list