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.<div><br>
</div><div>Thanks for the report! Keep any others coming!</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Sep 24, 2012 at 9:24 AM, David Dean <span dir="ltr"><<a href="mailto:david_dean@apple.com" target="_blank">david_dean@apple.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Fallout as requested:<br>
<br>
A crasher. Preprocessed source and shell script attached.<br>
<br>
llvm[6]: Compiling SubEngine.cpp for Release build<br>
Stack dump:<br>
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<br>

1.      <eof> parser at end of file<br>
2.      Per-module optimization passes<br>
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'.<br>

4.      Running pass 'SROA' on function '@_ZN5clang4ento12StoreManager13getLValueIvarEPKNS_12ObjCIvarDeclENS0_4SValE'<br>
clang: error: unable to execute command: Segmentation fault<br>
clang: error: clang frontend command failed due to signal (use -v to see invocation)<br>
<br>
<br><br>
On 23 Sep 2012, at 6:10 PM, Chandler Carruth <<a href="mailto:chandlerc@gmail.com">chandlerc@gmail.com</a>> wrote:<br>
<br>
> Author: chandlerc<br>
> Date: Sun Sep 23 20:10:25 2012<br>
> New Revision: 164480<br>
><br>
> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=164480&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=164480&view=rev</a><br>
> Log:<br>
> Enable the new SROA pass by default.<br>
><br>
> Queue the fallout. ;]<br>
><br>
> Modified:<br>
>    llvm/trunk/lib/Transforms/IPO/PassManagerBuilder.cpp<br>
><br>
> Modified: llvm/trunk/lib/Transforms/IPO/PassManagerBuilder.cpp<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Transforms/IPO/PassManagerBuilder.cpp?rev=164480&r1=164479&r2=164480&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Transforms/IPO/PassManagerBuilder.cpp?rev=164480&r1=164479&r2=164480&view=diff</a><br>

> ==============================================================================<br>
> --- llvm/trunk/lib/Transforms/IPO/PassManagerBuilder.cpp (original)<br>
> +++ llvm/trunk/lib/Transforms/IPO/PassManagerBuilder.cpp Sun Sep 23 20:10:25 2012<br>
> @@ -41,7 +41,7 @@<br>
>   cl::desc("Run GVN instead of Early CSE after vectorization passes"));<br>
><br>
> static cl::opt<bool> UseNewSROA("use-new-sroa",<br>
> -  cl::init(false), cl::Hidden,<br>
> +  cl::init(true), cl::Hidden,<br>
>   cl::desc("Enable the new, experimental SROA pass"));<br>
><br>
> PassManagerBuilder::PassManagerBuilder() {<br>
><br>
><br>
> _______________________________________________<br>
> llvm-commits mailing list<br>
> <a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
<br>
-David<br>
<br>
<br>
<br>_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
<br></blockquote></div><br></div>