[PATCH] D109624: [clang] Make the driver not diagnose errors on nonexistent linker inputs

2021-09-13 Thread Petr Hosek via Phabricator via cfe-commits
phosek added a comment. In D109624#2998774 , @saghir wrote: > In D109624#2998665 , @thakis wrote: > >> In D109624#2998550 , @saghir wrote: >> >>> Hi, this change also

[PATCH] D109624: [clang] Make the driver not diagnose errors on nonexistent linker inputs

2021-09-13 Thread Ahsan Saghir via Phabricator via cfe-commits
saghir added a comment. In D109624#2998665 , @thakis wrote: > In D109624#2998550 , @saghir wrote: > >> Hi, this change also breaks Power PC bots: >> >> 1. >>

[PATCH] D109624: [clang] Make the driver not diagnose errors on nonexistent linker inputs

2021-09-13 Thread Nico Weber via Phabricator via cfe-commits
thakis added a comment. In D109624#2998550 , @saghir wrote: > Hi, this change also breaks Power PC bots: > > 1. https://lab.llvm.org/buildbot/#/builders/19/builds/6451/steps/24/logs/stdio > 2.

[PATCH] D109624: [clang] Make the driver not diagnose errors on nonexistent linker inputs

2021-09-13 Thread Nico Weber via Phabricator via cfe-commits
thakis added a comment. In D109624#2998555 , @phosek wrote: > In D109624#2998419 , @thakis wrote: > >> Does it fail consistently? Or was it a one-time flake? > > It's been failing consistently. It might be

[PATCH] D109624: [clang] Make the driver not diagnose errors on nonexistent linker inputs

2021-09-13 Thread Nico Weber via Phabricator via cfe-commits
thakis added a comment. In D109624#2998550 , @saghir wrote: > Hi, this change also breaks Power PC bots: > > 1. https://lab.llvm.org/buildbot/#/builders/19/builds/6451/steps/24/logs/stdio > 2.

[PATCH] D109624: [clang] Make the driver not diagnose errors on nonexistent linker inputs

2021-09-13 Thread Petr Hosek via Phabricator via cfe-commits
phosek added a comment. In D109624#2998419 , @thakis wrote: > Does it fail consistently? Or was it a one-time flake? It's been failing consistently. It might be related to our toolchain setting lld as the default linker:

[PATCH] D109624: [clang] Make the driver not diagnose errors on nonexistent linker inputs

2021-09-13 Thread Ahsan Saghir via Phabricator via cfe-commits
saghir added a comment. Hi, this change also breaks Power PC bots: 1. https://lab.llvm.org/buildbot/#/builders/19/builds/6451/steps/24/logs/stdio 2. https://lab.llvm.org/buildbot/#/builders/18/builds/2443/steps/23/logs/stdio It fails consistently on our bots. Can you please take a look?

[PATCH] D109624: [clang] Make the driver not diagnose errors on nonexistent linker inputs

2021-09-13 Thread Nico Weber via Phabricator via cfe-commits
thakis added a comment. Does it fail consistently? Or was it a one-time flake? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D109624/new/ https://reviews.llvm.org/D109624 ___ cfe-commits mailing list

[PATCH] D109624: [clang] Make the driver not diagnose errors on nonexistent linker inputs

2021-09-13 Thread Nico Weber via Phabricator via cfe-commits
thakis added a comment. Hm, it passes on my win box and on the one running the presubmit test. Do you know if there's anything special about your bot? Any idea how I could reproduce this? The output is `error: command failed with exit status: True`. Repository: rG LLVM Github Monorepo

[PATCH] D109624: [clang] Make the driver not diagnose errors on nonexistent linker inputs

2021-09-13 Thread Petr Hosek via Phabricator via cfe-commits
phosek added a comment. `Clang :: Driver/cl.c` started failing on our Windows bots after this change landed. Here

[PATCH] D109624: [clang] Make the driver not diagnose errors on nonexistent linker inputs

2021-09-13 Thread Nico Weber via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG648feabc65d8: [clang] Make the driver not diagnose errors on nonexistent linker inputs (authored by thakis). Herald added a project: clang. Herald added a subscriber: cfe-commits. Repository: rG LLVM