This is an archive of the discontinued LLVM Phabricator instance.

[test][clang] run test for lld emitting dwarf-aranages only if lld is presented
ClosedPublic

Authored by azat on Sep 14 2022, 12:22 AM.

Diff Detail

Event Timeline

azat created this revision.Sep 14 2022, 12:22 AM
Herald added a project: Restricted Project. · View Herald TranscriptSep 14 2022, 12:22 AM
azat requested review of this revision.Sep 14 2022, 12:22 AM
Herald added a project: Restricted Project. · View Herald TranscriptSep 14 2022, 12:22 AM
Herald added a subscriber: cfe-commits. · View Herald Transcript
azat retitled this revision from [test][clang] run test for lld emitting dwarf-aranages only if lld is presented Fixes: https://reviews.llvm.org/D133092 CI: https://lab.llvm.org/buildbot/#/builders/109/builds/46592 to [test][clang] run test for lld emitting dwarf-aranages only if lld is presented.Sep 14 2022, 12:23 AM
azat edited the summary of this revision. (Show Details)
hokein accepted this revision.Sep 14 2022, 12:59 AM
hokein added a subscriber: hokein.

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@).

This revision is now accepted and ready to land.Sep 14 2022, 12:59 AM
This revision was landed with ongoing or failed builds.Sep 14 2022, 1:17 AM
This revision was automatically updated to reflect the committed changes.
alexgatea reopened this revision.Sep 14 2022, 5:33 AM
alexgatea added a subscriber: alexgatea.

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?

This revision is now accepted and ready to land.Sep 14 2022, 5:33 AM
azat added a comment.Sep 14 2022, 5:38 AM

Can you please fix this?

It should be fixed in https://reviews.llvm.org/D133847

alexgatea closed this revision.Sep 14 2022, 5:42 AM

It should be fixed in https://reviews.llvm.org/D133847

Ah I didn't see that. Thank you!

thakis added a subscriber: thakis.Sep 14 2022, 9:17 AM

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.

azat added a comment.Sep 14 2022, 9:25 AM

@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