[PATCH] D143501: [clang][DebugInfo] lldb: Use preferred name's type when emitting DW_AT_names

Michael Buch via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Feb 9 08:02:49 PST 2023


Michael137 added inline comments.


================
Comment at: lldb/test/API/functionalities/data-formatter/data-formatter-stl/libcxx/shared_ptr/TestDataFormatterLibcxxSharedPtr.py:61
 
         if self.expectedCompilerVersion(['>', '16.0']):
+            string_type = "std::string"
----------------
Michael137 wrote:
> aprantl wrote:
> > Out of curiosity: How does this function work when building the testsuite with a compiler that isn't Clang?
> > I vaguely recall there was someone in the community running a bot that built the LLDB testsuite against GCC.
> Good point, this should also check for the compiler really. I assume because GCC isn't at that version yet it just works on that GCC buildbot
> 
> Will change in a different patch since there's other tests with this condition
Addressed here: https://reviews.llvm.org/D143656


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D143501



More information about the cfe-commits mailing list