[PATCH] D151001: [DebugInfo][NFCI] Add unittest for DWARFAbbreviationDeclarationSet
Alex Langford via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue May 23 10:22:57 PDT 2023
bulbazord added a comment.
In D151001#4363438 <https://reviews.llvm.org/D151001#4363438>, @jhenderson wrote:
> It would have been better to wait until I'd had a chance to review your latest updates, as there are a few (admittedly minor) points I'd have picked up on that now need you to go and update the in-tree code. I'm also adding a note that I haven't reviewed the coverage of these tests, just the general approach and style.
Apologies, I was too eager to land this. I've uploaded a new phabricator diff to address your comments. We can continue iterating there, sorry about this. D151233 <https://reviews.llvm.org/D151233>
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D151001/new/
https://reviews.llvm.org/D151001
More information about the llvm-commits
mailing list