[Lldb-commits] [PATCH] D145569: [lldb][InstrumentationRuntime] Make 'data' struct anonymous in order to avoid collisions with types in the debuggee

[Jim Ingham via Phabricator via lldb-commits]

jingham added a comment.

The TSAN instrumenter has the same `struct data` definition.  I actually got a report of this crashing in TSAN yesterday.  You should do that one as well.  Otherwise, LGTM and I don't think you need to add a test with "struct data" in it on the off chance that someone reverts this back to exactly `struct data`...


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D145569