This is an archive of the discontinued LLVM Phabricator instance.

Adding GDB PrettyPrinter for mlir::Identifier.
ClosedPublic

Authored by csigg on Sep 1 2020, 6:51 AM.

Details

Summary

This is the first bit from D73546. Primarily setting up the corresponding test. Will add more pretty printers in a separate revision.

Diff Detail

Event Timeline

csigg created this revision.Sep 1 2020, 6:51 AM
Herald added a project: Restricted Project. · View Herald TranscriptSep 1 2020, 6:51 AM
csigg requested review of this revision.Sep 1 2020, 6:51 AM
csigg edited the summary of this revision. (Show Details)Sep 2 2020, 12:37 PM
csigg added a reviewer: dblaikie.
csigg added a comment.Sep 2 2020, 12:39 PM

I think clang-tidy can't find the include because there is no complication database for that file because we don't run debuginfo-tests as part of the premerge-check. We can either change that (but dexter tests fail for me locally), add the file to .clang-tidy-ignore, or simply ignore the error. Which one would you prefer?

Ignoring clang-tidy is the common solution here I believe.

dblaikie accepted this revision.Sep 2 2020, 1:51 PM

Generally looks plausible to me.

This revision is now accepted and ready to land.Sep 2 2020, 1:51 PM
This revision was landed with ongoing or failed builds.Sep 2 2020, 11:18 PM
This revision was automatically updated to reflect the committed changes.
Orlando added a subscriber: Orlando.Sep 3 2020, 2:55 AM

I've reverted this for now (485e6db8729) because one of our internal build bot is failing in the same way. I don't think many (any?) of the bots on http://lab.llvm.org:8011/console build the debuginfo-tests project!