[PATCH] D157808: [clang] Add missing field to VisibilityAttr json AST dump

Tue Aug 15 06:56:45 PDT 2023

thakis added a comment.

In D157808#4587014 <https://reviews.llvm.org/D157808#4587014>, @thakis wrote:

> Hello, one of the many dump commits broke tests on macOS: http://45.33.8.238/macm1/67030/step_7.txt
>
> Please take a look and revert for now if it takes a while to fix.
>
> Also, consider spreading out commits a bit so that if one breaks something, it's easily to see which one it was.

Turns out the failure is from D157775 <https://reviews.llvm.org/D157775>. Tests have been broken on mac for over 13 hours now. Could you please take a look?

I'll revert D157775 <https://reviews.llvm.org/D157775> and follow-ups causing conflicts if it's still broken in another 2h.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D157808