Details
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
This looks good to me, I'm going to land this for you once I confirm that it fixes the test failure (we can address any post comments from dblaikie@).
Hi
This test case is still failing for me due to the lines without -fuse-ld=ldd:
// RUN: %clang -### -target x86_64-unknown-linux -flto -gdwarf-aranges %s 2>&1 | FileCheck -check-prefix=LDGARANGE %s // RUN: %clang -### -target x86_64-unknown-linux -flto=thin -gdwarf-aranges %s 2>&1 | FileCheck -check-prefix=LDGARANGE %s
Can you please fix this?
It should be fixed in https://reviews.llvm.org/D133847
Ah I didn't see that. Thank you!
the test crashes with sanitizers https://lab.llvm.org/buildbot/#/builders/5/builds/27437
the test crashes with sanitizers https://lab.llvm.org/buildbot/#/builders/5/builds/27437
yep, this should be fixed in https://reviews.llvm.org/D133847
As far as I can tell, this broke check-clang everywhere. (See e.g.: https://lab.llvm.org/buildbot/#/builders/139/builds/28039 and many of the bots on https://lab.llvm.org/buildbot/#/console
Before landing changes, please:
- run tests locally
- wait for the presubmit bots to post a success message on your phab review
After landing changes, please:
- watch the bots
- if they turn red, either quickly fix or quickly revert
Please don't land a change and then keep bots red for over 8 hours.
@thakis Thanks for taking a look!
Actually initially the problem had been introduced in https://reviews.llvm.org/D133092
I've run tests locally, however I did have lld in a system, so I did not noticed the problem.
And CI did not find the problem either, since I guess it also has lld.
Final fix for this problem should be - https://reviews.llvm.org/D133847