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 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
