[PATCH] D92028: Fix driver test from e16c0a9a689719

2020-11-24 Thread David Zarzycki via Phabricator via cfe-commits
davezarzycki added a comment. This is breaking building on Fedora 33 (x86-64). Can we revert this or get a quick fix? -- Exit Code: 1 Command Output (stdout): -- "/tmp/_update_lc/r/bin/ld64.lld" "-demangle" "-dynamic" "-arch" "i386" "-macosx_version_min" "10.5.0" "-o"

[PATCH] D92028: Fix driver test from e16c0a9a689719

2020-11-24 Thread Nico Weber via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG5ce85e66358a: Fix driver test from e16c0a9a689719 (authored by thakis). Herald added a project: clang. Changed prior to commit: https://reviews.llvm.org/D92028?vs=307335=307361#toc Repository: rG

[PATCH] D92028: Fix driver test from e16c0a9a689719

2020-11-24 Thread Hans Wennborg via Phabricator via cfe-commits
hans accepted this revision. hans added inline comments. This revision is now accepted and ready to land. Comment at: clang/test/Driver/darwin-ld-demangle-lld.c:11 +// RUN: | FileCheck %s +// CHECK: "-demangle" nit: most tests have a blank line between the RUN

[PATCH] D92028: Fix driver test from e16c0a9a689719

2020-11-24 Thread Nico Weber via Phabricator via cfe-commits
thakis created this revision. thakis added a reviewer: hans. thakis requested review of this revision. The test failed silently if lld wasn't built alongside clang. But the test uses -###, so the "invalid linker name in -fuse-ld=lld" diag didn't make clang fail, and something else happened to