<html><head><meta http-equiv="Content-Type" content="text/html; charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class="">IIRC, the dwarf version on Darwin depends on the deployment target and is 2 for earlier versions of macOS and and 4 for newer ones (I need to dig through the driver code to determine the exact version).<div class=""><br class=""></div><div class="">I can take a look at this tomorrow morning.</div><div class=""><br class=""></div><div class="">-- adrian<br class=""><div class=""><div><br class=""><blockquote type="cite" class=""><div class="">On Apr 4, 2019, at 6:21 PM, Alex L <<a href="mailto:arphaman@gmail.com" class="">arphaman@gmail.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div dir="ltr" class=""><div dir="ltr" class=""><div dir="ltr" class=""><div dir="ltr" class="">Hi Stephen,<div class=""><br class=""></div><div class="">It looks like your change has caused the 'debug-options.c' test to start failing on Darwin. It looks like we emit "-dwarf-version=4" by default, and not 2. Could you please take a look.</div><div class=""><br class=""></div><div class="">Here's a link to a latest build on the public LLVM CI that contains the failure:</div><div class=""><a href="http://lab.llvm.org:8080/green/job/clang-stage1-cmake-RA-incremental/59772/console" class="">http://lab.llvm.org:8080/green/job/clang-stage1-cmake-RA-incremental/59772/console</a><br class=""></div><div class=""><br class=""></div><div class="">Here's the failure itself:</div><div class=""><span style="color:rgb(51,51,51);font-family:monospace;font-size:13px;white-space:pre-wrap" class="">Command Output (stderr):
--
</span><a id="gmail-15553146749ba4694-19c4-4d7e-bec5-911270d8a58c" style="box-sizing:border-box;word-wrap:break-word;color:rgb(51,51,51);font-family:monospace;font-size:13px;white-space:pre-wrap;display:block" class=""></a><span title="Compile Error" style="box-sizing:border-box;font-family:monospace;font-size:13px;white-space:pre-wrap;color:white;background-color:red" class="">/Users/buildslave/jenkins/workspace/clang-stage1-cmake-RA-incremental/llvm/tools/clang/test/Driver/debug-options.c:280:14: error: G_DWARF2: expected string not found in input
</span><span style="color:rgb(51,51,51);font-family:monospace;font-size:13px;white-space:pre-wrap" class="">// G_DWARF2: "-dwarf-version=2"
^
<stdin>:5:515: note: scanning from here
"/Users/buildslave/jenkins/workspace/clang-stage1-cmake-RA-incremental/clang-build/bin/clang-9" "-cc1" "-triple" "x86_64-apple-macosx10.13.0" "-Wdeprecated-objc-isa-usage" "-Werror=deprecated-objc-isa-usage" "-emit-obj" "-mrelax-all" "-disable-free" "-main-file-name" "debug-options.c" "-mrelocation-model" "pic" "-pic-level" "2" "-mthread-model" "posix" "-mdisable-fp-elim" "-masm-verbose" "-munwind-tables" "-faligned-alloc-unavailable" "-target-cpu" "penryn" "-dwarf-column-info" "-debug-info-kind=standalone" "-dwarf-version=4" "-debugger-tuning=lldb" "-ggnu-pubnames" "-target-linker-version" "351.8" "-coverage-notes-file" "/Users/buildslave/jenkins/workspace/clang-stage1-cmake-RA-incremental/clang-build/tools/clang/test/Driver/debug-options.gcno" "-resource-dir" "/Users/buildslave/jenkins/workspace/clang-stage1-cmake-RA-incremental/clang-build/lib/clang/9.0.0" "-fdebug-compilation-dir" "/Users/buildslave/jenkins/workspace/clang-stage1-cmake-RA-incremental/clang-build/tools/clang/test/Driver" "-ferror-limit" "19" "-fmessage-length" "0" "-stack-protector" "1" "-fblocks" "-fencode-extended-block-signature" "-fregister-global-dtors-with-atexit" "-fobjc-runtime=macosx-10.13.0" "-fmax-type-align=16" "-fdiagnostics-show-option" "-o" "debug-options.o" "-x" "c" "/Users/buildslave/jenkins/workspace/clang-stage1-cmake-RA-incremental/llvm/tools/clang/test/Driver/debug-options.c"
^</span><br class=""></div><div class=""><br class=""></div><div class=""><br class=""></div><div class="">Adrian, I've looped you in since you reviewed the commit it looks like. Do you know if that's expected behavior on Darwin?</div><div class=""><br class=""></div><div class="">Cheers,</div><div class="">Alex</div></div></div></div></div><br class=""><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, 4 Apr 2019 at 11:16, Stephen Hines via cfe-commits <<a href="mailto:cfe-commits@lists.llvm.org" class="">cfe-commits@lists.llvm.org</a>> wrote:<br class=""></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">Author: srhines<br class="">
Date: Thu Apr 4 11:17:46 2019<br class="">
New Revision: 357713<br class="">
<br class="">
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=357713&view=rev" rel="noreferrer" target="_blank" class="">http://llvm.org/viewvc/llvm-project?rev=357713&view=rev</a><br class="">
Log:<br class="">
Verify that Android targets generate DWARF 4 by default.<br class="">
<br class="">
Summary:<br class="">
In the future, Android releases will support DWARF 5, but we need to<br class="">
ensure that older targets only have DWARF 4 generated for them. This<br class="">
patch inserts that verification for all Android releases now. The patch<br class="">
also fixes 2 minor mistakes (a mistakenly moved RUN line, and the<br class="">
missing G_DWARF2 check label).<br class="">
<br class="">
Reviewers: aprantl<br class="">
<br class="">
Reviewed By: aprantl<br class="">
<br class="">
Subscribers: chh, pirama, cfe-commits<br class="">
<br class="">
Tags: #clang<br class="">
<br class="">
Differential Revision: <a href="https://reviews.llvm.org/D60238" rel="noreferrer" target="_blank" class="">https://reviews.llvm.org/D60238</a><br class="">
<br class="">
Modified:<br class="">
cfe/trunk/test/Driver/debug-options.c<br class="">
<br class="">
Modified: cfe/trunk/test/Driver/debug-options.c<br class="">
URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Driver/debug-options.c?rev=357713&r1=357712&r2=357713&view=diff" rel="noreferrer" target="_blank" class="">http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Driver/debug-options.c?rev=357713&r1=357712&r2=357713&view=diff</a><br class="">
==============================================================================<br class="">
--- cfe/trunk/test/Driver/debug-options.c (original)<br class="">
+++ cfe/trunk/test/Driver/debug-options.c Thu Apr 4 11:17:46 2019<br class="">
@@ -17,9 +17,14 @@<br class="">
// RUN: %clang -### -c -glldb %s -target x86_64-linux-gnu 2>&1 \<br class="">
// RUN: | FileCheck -check-prefix=G -check-prefix=G_LLDB %s<br class="">
// RUN: %clang -### -c -gsce %s -target x86_64-linux-gnu 2>&1 \<br class="">
+// RUN: | FileCheck -check-prefix=G -check-prefix=G_SCE %s<br class="">
+<br class="">
+// Android.<br class="">
+// Android should always generate DWARF4.<br class="">
+// RUN: %clang -### -c -g %s -target arm-linux-androideabi 2>&1 \<br class="">
+// RUN: | FileCheck -check-prefix=G -check-prefix=G_DWARF4 %s<br class="">
<br class="">
// Darwin.<br class="">
-// RUN: | FileCheck -check-prefix=G -check-prefix=G_SCE %s<br class="">
// RUN: %clang -### -c -g %s -target x86_64-apple-darwin 2>&1 \<br class="">
// RUN: | FileCheck -check-prefix=G_STANDALONE \<br class="">
// RUN: -check-prefix=G_DWARF2 \<br class="">
@@ -272,6 +277,7 @@<br class="">
//<br class="">
// G_STANDALONE: "-cc1"<br class="">
// G_STANDALONE: "-debug-info-kind=standalone"<br class="">
+// G_DWARF2: "-dwarf-version=2"<br class="">
// G_DWARF4: "-dwarf-version=4"<br class="">
//<br class="">
// G_GDB: "-debugger-tuning=gdb"<br class="">
<br class="">
<br class="">
_______________________________________________<br class="">
cfe-commits mailing list<br class="">
<a href="mailto:cfe-commits@lists.llvm.org" target="_blank" class="">cfe-commits@lists.llvm.org</a><br class="">
<a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits" rel="noreferrer" target="_blank" class="">https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits</a><br class="">
</blockquote></div>
</div></blockquote></div><br class=""></div></div></body></html>