[PATCH] D105457: [clang] Refactor AST printing tests to share more infrastructure

Wed Jul 14 15:53:30 PDT 2021

dyung accepted this revision.
dyung added a comment.

In D105457#2878352 <https://reviews.llvm.org/D105457#2878352>, @nridge wrote:

> In D105457#2878302 <https://reviews.llvm.org/D105457#2878302>, @dyung wrote:
>
>> Interesting. If you can get me an updated patch, I can give it a try on my machine with 7.5 to verify if you like.
>
> I posted an updated patch, if you can give that a whirl that would be great!

I can confirm that it builds and tests successfully on a machine with gcc 7.5. Assuming you have tested with more recent compilers and that it is fine there, LGTM.

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D105457/new/

https://reviews.llvm.org/D105457