[Lldb-commits] [PATCH] D149394: Finish the job of D145569

Jim Ingham via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Thu Apr 27 17:25:59 PDT 2023


jingham created this revision.
jingham added reviewers: Michael137, JDevlieghere.
Herald added a project: All.
jingham requested review of this revision.
Herald added a project: LLDB.
Herald added a subscriber: lldb-commits.

The  InstrumentationRuntime's defined a data structure to gather report data, and repeated the pattern of defining it like:

struct data {
};
data t = {0};

That causes conflicts against other types called data, and could cause the expression not to compile.  Michael fixed this for some of the sanitizers in:

https://reviews.llvm.org/D145569

but neglected to do it for the thread sanitizer.

This patch adds the fix to that sanitizer.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D149394

Files:
  lldb/source/Plugins/InstrumentationRuntime/TSan/InstrumentationRuntimeTSan.cpp


Index: lldb/source/Plugins/InstrumentationRuntime/TSan/InstrumentationRuntimeTSan.cpp
===================================================================
--- lldb/source/Plugins/InstrumentationRuntime/TSan/InstrumentationRuntimeTSan.cpp
+++ lldb/source/Plugins/InstrumentationRuntime/TSan/InstrumentationRuntimeTSan.cpp
@@ -94,7 +94,7 @@
 const int REPORT_TRACE_SIZE = 128;
 const int REPORT_ARRAY_SIZE = 4;
 
-struct data {
+struct {
     void *report;
     const char *description;
     int report_count;


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D149394.517758.patch
Type: text/x-patch
Size: 507 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20230428/3bd53e39/attachment.bin>


More information about the lldb-commits mailing list