<html><head><meta http-equiv="Content-Type" content="text/html charset=windows-1252"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;"><br><div><div>On Nov 12, 2013, at 8:27 AM, jahanian <<a href="mailto:fjahanian@apple.com">fjahanian@apple.com</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><br>On Nov 11, 2013, at 6:28 PM, Richard Smith <<a href="mailto:richard@metafoo.co.uk">richard@metafoo.co.uk</a>> wrote:<br><br><blockquote type="cite">This is still failing for me. Here's the command like I get:<br><br> "/mnt/space/build/llvm-3/build/bin/clang-3.4" "-cc1" "-triple" "x86_64-unknown-linux-gnu" "-rewrite-objc" "-disable-free" "-main-file-name" "rewrite-legacy-objc.m" "-mrelocation-model" "static" "-mdisable-fp-elim" "-fmath-errno" "-masm-verbose" "-mconstructor-aliases" "-munwind-tables" "-target-cpu" "x86-64" "-resource-dir" "/mnt/space/build/llvm-3/build/bin/../lib/clang/3.4" "-internal-isystem" "/usr/local/include" "-internal-isystem" "/mnt/space/build/llvm-3/build/bin/../lib/clang/3.4/include" "-internal-externc-isystem" "/usr/include/x86_64-linux-gnu" "-internal-externc-isystem" "/include" "-internal-externc-isystem" "/usr/include" "-fdebug-compilation-dir" "/mnt/space/build/llvm-3/build/tools/clang/test/Driver" "-ferror-limit" "19" "-fmessage-length" "0" "-mstackrealign" "-fobjc-runtime=macosx-fragile" "-fobjc-subscripting-legacy-runtime" "-fno-objc-infer-related-result-type" "-fobjc-exceptions" "-fdiagnostics-show-option" "-vectorize-slp" "-o" "-" "-x" "objective-c" "/home/richardsmith/llvm-3/src/tools/clang/test/Driver/rewrite-legacy-objc.m"<br><br>There's no -stack-protector in there, and you're not going to get -fblocks on my system, because it's not supported on this target. Also in the output:<br><br>clang-3.4: warning: argument unused during compilation: '-arch i386'<br><br>Is this test missing a -target, perhaps?<br><br><br></blockquote><br>Yes. Need to pick up the right “-target” Looks like the test has been reverted.<br></blockquote><div><br></div>Reapplied patch in r<span style="font-family: Menlo; font-size: 11px;">194489 </span> with the modified test to use -target.</div><div><br></div><div>- Fariborz</div><div><br><blockquote type="cite"><br>Will take a look.<br>- Fariborz<br><br><br><br><br>_______________________________________________<br>cfe-commits mailing list<br><a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits<br></blockquote></div><br></body></html>