Revert "[Sanitizers] Remove BuildId from sanitizers stacktrace on
authorFlorian Mayer <fmayer@google.com>
Wed, 7 Jun 2023 00:25:55 +0000 (17:25 -0700)
committerFlorian Mayer <fmayer@google.com>
Wed, 7 Jun 2023 00:25:55 +0000 (17:25 -0700)
Breaks tests.

This reverts commit fdb1a891b64c27522a2386a8025f8ad5c7e02bfb.

compiler-rt/lib/sanitizer_common/sanitizer_stacktrace_printer.cpp
compiler-rt/lib/sanitizer_common/tests/sanitizer_stacktrace_printer_test.cpp

index d3d1d26..1096d21 100644 (file)
@@ -218,9 +218,7 @@ void RenderFrame(InternalScopedString *buffer, const char *format, int frame_no,
         RenderModuleLocation(buffer, info->module, info->module_offset,
                              info->module_arch, strip_path_prefix);
 
-#ifndef SANITIZER_APPLE
         MaybeBuildIdToBuffer(*info, /*PrefixSpace=*/true, buffer);
-#endif
       } else {
         buffer->append("(<unknown module>)");
       }
@@ -233,9 +231,7 @@ void RenderFrame(InternalScopedString *buffer, const char *format, int frame_no,
         // Always strip the module name for %M.
         RenderModuleLocation(buffer, StripModuleName(info->module),
                              info->module_offset, info->module_arch, "");
-#ifndef SANITIZER_APPLE
         MaybeBuildIdToBuffer(*info, /*PrefixSpace=*/true, buffer);
-#endif
       } else {
         buffer->append("(%p)", (void *)address);
       }
index 489ef4d..62b34cd 100644 (file)
@@ -137,19 +137,11 @@ TEST(SanitizerStacktracePrinter, RenderFrame) {
   RenderFrame(&str, "%M", frame_no, info.address, &info, false);
   EXPECT_NE(nullptr, internal_strstr(str.data(), "(module+0x"));
   EXPECT_NE(nullptr, internal_strstr(str.data(), "200"));
-#if SANITIZER_APPLE
-  EXPECT_EQ(nullptr, internal_strstr(str.data(), "BuildId: 5566"));
-#else
   EXPECT_NE(nullptr, internal_strstr(str.data(), "BuildId: 5566"));
-#endif
   str.clear();
 
   RenderFrame(&str, "%L", frame_no, info.address, &info, false);
-#if SANITIZER_APPLE
-  EXPECT_STREQ("(/path/to/module+0x200)", str.data());
-#else
   EXPECT_STREQ("(/path/to/module+0x200) (BuildId: 5566)", str.data());
-#endif
   str.clear();
 
   RenderFrame(&str, "%b", frame_no, info.address, &info, false);