r326168 - Attempt to fix greendragon bot after r326141

Mon Feb 26 23:22:18 PST 2018

Hmm. Reid had included a fix for compiler-rt in r326141, but that doesn't seem particularly relevant to this failure. In any case, I expect I'll receive emails when that bot is up and running again, and I can take a look then. (It didn't repro locally on Linux for me.)

From: Zachary Turner <zturner at google.com>
Date: Monday, February 26, 2018 at 10:36 PM
To: Adam Nemet <anemet at apple.com>
Cc: Shoaib Meenai <smeenai at fb.com>, "cfe-commits at lists.llvm.org" <cfe-commits at lists.llvm.org>, Reid Kleckner <rnk at google.com>
Subject: Re: r326168 - Attempt to fix greendragon bot after r326141

So this doesn’t appear to fix the issue that led to my revert earlier.

http://lab.llvm.org:8011/builders/clang-cmake-x86_64-sde-avx512-linux/builds/6760<https://urldefense.proofpoint.com/v2/url?u=http-3A__lab.llvm.org-3A8011_builders_clang-2Dcmake-2Dx86-5F64-2Dsde-2Davx512-2Dlinux_builds_6760&d=DwMFaQ&c=5VD0RTtNlTh3ycd41b3MUw&r=o3kDXzdBUE3ljQXKeTWOMw&m=5wMAQAwunEFDagi_DAaHOfwi-PPxp58tkg-yxq-vXlg&s=SAVebUMWaqXvTSR_3fEb944vR-Gk4dliL4EUOVgJkho&e=>

There’s no slashes in the command line that’s failing. However, this bot doesn’t seem to be updating and hasn’t pulled in this cl yet, but I expect it to fail again when it does
On Mon, Feb 26, 2018 at 10:27 PM Zachary Turner <zturner at google.com<mailto:zturner at google.com>> wrote:
Never mind, I didn’t realize Reid resubmitted my cl after I reverted it. In fact, I reverted it because of this greendragon failure which I wasn’t sure how to fix at the time
On Mon, Feb 26, 2018 at 10:12 PM Adam Nemet <anemet at apple.com<mailto:anemet at apple.com>> wrote:
BTW, I think that the LLVM bots have trouble accessing subversion too.  I already let Galina know.



On Feb 26, 2018, at 10:06 PM, Shoaib Meenai <smeenai at fb.com<mailto:smeenai at fb.com>> wrote:

r326171 works locally. I'll keep an eye on the bots.

From: <anemet at apple.com<mailto:anemet at apple.com>> on behalf of Adam Nemet <anemet at apple.com<mailto:anemet at apple.com>>
Date: Monday, February 26, 2018 at 9:51 PM
To: Shoaib Meenai <smeenai at fb.com<mailto:smeenai at fb.com>>
Cc: Zachary Turner <zturner at google.com<mailto:zturner at google.com>>, "cfe-commits at lists.llvm.org<mailto:cfe-commits at lists.llvm.org>" <cfe-commits at lists.llvm.org<mailto:cfe-commits at lists.llvm.org>>, Reid Kleckner <rnk at google.com<mailto:rnk at google.com>>
Subject: Re: r326168 - Attempt to fix greendragon bot after r326141

Yep.

On Feb 26, 2018, at 9:50 PM, Shoaib Meenai <smeenai at fb.com<mailto:smeenai at fb.com>> wrote:

Reid re-committed r326141 this morning (and followed up with r326144), and I don't think either of those have been reverted?

From: Zachary Turner <zturner at google.com<mailto:zturner at google.com>>
Date: Monday, February 26, 2018 at 9:47 PM
To: Adam Nemet <anemet at apple.com<mailto:anemet at apple.com>>
Cc: Shoaib Meenai <smeenai at fb.com<mailto:smeenai at fb.com>>, "cfe-commits at lists.llvm.org<mailto:cfe-commits at lists.llvm.org>" <cfe-commits at lists.llvm.org<mailto:cfe-commits at lists.llvm.org>>, Reid Kleckner <rnk at google.com<mailto:rnk at google.com>>
Subject: Re: r326168 - Attempt to fix greendragon bot after r326141

I already reverted this a long time ago, but if Shoaib has a proper fix that would be great
On Mon, Feb 26, 2018 at 9:45 PM Adam Nemet <anemet at apple.com<mailto:anemet at apple.com>> wrote:
Ah, that should be sufficient.



On Feb 26, 2018, at 9:44 PM, Shoaib Meenai <smeenai at fb.com<mailto:smeenai at fb.com>> wrote:

Thanks. I'm building on macOS locally to confirm the original problem and my fix.

From: <anemet at apple.com<mailto:anemet at apple.com>> on behalf of Adam Nemet <anemet at apple.com<mailto:anemet at apple.com>>
Date: Monday, February 26, 2018 at 9:42 PM
To: Shoaib Meenai <smeenai at fb.com<mailto:smeenai at fb.com>>
Cc: "cfe-commits at lists.llvm.org<mailto:cfe-commits at lists.llvm.org>" <cfe-commits at lists.llvm.org<mailto:cfe-commits at lists.llvm.org>>, Reid Kleckner <rnk at google.com<mailto:rnk at google.com>>, Zachary Turner <zturner at google.com<mailto:zturner at google.com>>
Subject: Re: r326168 - Attempt to fix greendragon bot after r326141

This is the bot with the failure:

http://lab.llvm.org:8080/green/job/clang-stage1-cmake-RA-incremental/46794/#showFailuresLink<https://urldefense.proofpoint.com/v2/url?u=http-3A__lab.llvm.org-3A8080_green_job_clang-2Dstage1-2Dcmake-2DRA-2Dincremental_46794_-23showFailuresLink&d=DwMFaQ&c=5VD0RTtNlTh3ycd41b3MUw&r=o3kDXzdBUE3ljQXKeTWOMw&m=OIgCAeO_dFHAJFlsBo7l_qhVKV7niFfyQ0fpICifvN4&s=OEMtPu3dUHXK8327qnfWOSUP62v1F-s_a13X0lqbdJ0&e=>

But to make matters worse, greendragon is experience some difficulty fetching from SVN so you might want to wait until things recover.  I am not sure if I will be able to get in touch with Mike Edwards until the morning PST.

There may also be llvm bots that exhibit the same problem.  You may have better luck with those in the short term.

Adam

On Feb 26, 2018, at 9:35 PM, Shoaib Meenai <smeenai at fb.com<mailto:smeenai at fb.com>> wrote:

Could you point me to the specific bot that was failing? I've dealt with that problem before, and it shouldn't be hard to fix, but I wanna monitor the bot after the fix to make sure it stays green.

From: <anemet at apple.com<mailto:anemet at apple.com>> on behalf of Adam Nemet <anemet at apple.com<mailto:anemet at apple.com>>
Date: Monday, February 26, 2018 at 9:30 PM
To: Shoaib Meenai <smeenai at fb.com<mailto:smeenai at fb.com>>
Cc: "cfe-commits at lists.llvm.org<mailto:cfe-commits at lists.llvm.org>" <cfe-commits at lists.llvm.org<mailto:cfe-commits at lists.llvm.org>>, Reid Kleckner <rnk at google.com<mailto:rnk at google.com>>, Zachary Turner <zturner at google.com<mailto:zturner at google.com>>
Subject: Re: r326168 - Attempt to fix greendragon bot after r326141

I don’t think we can deal with the slash options:

clang-7.0: warning: '/Users/buildslave/jenkins/workspace/apple-clang-master-RA-stage1-cmake-incremental/clang/src/tools/clang/test/Driver/codeview-column-info.c' treated as the '/U' option [-Wslash-u-filename]
clang-7.0: note: Use '--' to treat subsequent arguments as filenames
clang-7.0: warning: argument unused during compilation: '/Z7' [-Wunused-command-line-argument]
clang-7.0: warning: argument unused during compilation: ‘-U sers/buildslave/jenkins/workspace/apple-clang-master-RA-stage1-cmake-incremental/clang/src/tools/clang/test/Driver/codeview-column-info.c' [-Wunused-command-line-argument]

Anyhow as I said to Reid, feel free to improve the patch, I am just bringing back a bot that has been red for hours.

Adam


On Feb 26, 2018, at 9:22 PM, Shoaib Meenai <smeenai at fb.com<mailto:smeenai at fb.com>> wrote:

This seems bogus to me. CodeView can be generated on any build platform; it just needs the correct triple, which Reid added in r326144.

From: cfe-commits <cfe-commits-bounces at lists.llvm.org<mailto:cfe-commits-bounces at lists.llvm.org>> on behalf of Adam Nemet via cfe-commits <cfe-commits at lists.llvm.org<mailto:cfe-commits at lists.llvm.org>>
Reply-To: Adam Nemet <anemet at apple.com<mailto:anemet at apple.com>>
Date: Monday, February 26, 2018 at 8:51 PM
To: "cfe-commits at lists.llvm.org<mailto:cfe-commits at lists.llvm.org>" <cfe-commits at lists.llvm.org<mailto:cfe-commits at lists.llvm.org>>
Subject: r326168 - Attempt to fix greendragon bot after r326141

Author: anemet
Date: Mon Feb 26 20:49:26 2018
New Revision: 326168

URL: https://urldefense.proofpoint.com/v2/url?u=http-3A__llvm.org_viewvc_llvm-2Dproject-3Frev-3D326168-26view-3Drev&d=DwIGaQ&c=5VD0RTtNlTh3ycd41b3MUw&r=o3kDXzdBUE3ljQXKeTWOMw&m=B6WfLLVLbYMU_571sI0XgBTcOm561QyDCKF2UOJyc-k&s=oqoKnyrAT6kNwxIasdWb7eopGd0q41TFJ5Hxp_eoiZs&e=
Log:
Attempt to fix greendragon bot after r326141

Modified:
    cfe/trunk/test/Driver/codeview-column-info.c

Modified: cfe/trunk/test/Driver/codeview-column-info.c
URL: https://urldefense.proofpoint.com/v2/url?u=http-3A__llvm.org_viewvc_llvm-2Dproject_cfe_trunk_test_Driver_codeview-2Dcolumn-2Dinfo.c-3Frev-3D326168-26r1-3D326167-26r2-3D326168-26view-3Ddiff&d=DwIGaQ&c=5VD0RTtNlTh3ycd41b3MUw&r=o3kDXzdBUE3ljQXKeTWOMw&m=B6WfLLVLbYMU_571sI0XgBTcOm561QyDCKF2UOJyc-k&s=Hs37wkn2y2OFh6b_DWdNuK_AD--vySi-ptv_d_HDqxg&e=
==============================================================================

--- cfe/trunk/test/Driver/codeview-column-info.c (original)
+++ cfe/trunk/test/Driver/codeview-column-info.c Mon Feb 26 20:49:26 2018
@@ -1,3 +1,4 @@
+// REQUIRES: system-windows
// Check that -dwarf-column-info does not get added to the cc1 line:
// 1) When -gcodeview is present via the clang or clang++ driver
// 2) When /Z7 is present via the cl driver.


_______________________________________________
cfe-commits mailing list
cfe-commits at lists.llvm.org<mailto:cfe-commits at lists.llvm.org>
https://urldefense.proofpoint.com/v2/url?u=http-3A__lists.llvm.org_cgi-2Dbin_mailman_listinfo_cfe-2Dcommits&d=DwIGaQ&c=5VD0RTtNlTh3ycd41b3MUw&r=o3kDXzdBUE3ljQXKeTWOMw&m=B6WfLLVLbYMU_571sI0XgBTcOm561QyDCKF2UOJyc-k&s=JTIVfh7ogIVoFYMFdjOK-30TFLWLX39y8q9KItC_xnY&e=

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20180227/145e7fc9/attachment-0001.html>
