[Lldb-commits] [PATCH] D136171: [lldb-tests] Remove dubious standard library flag

Felipe de Azevedo Piovezan via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Tue Oct 18 06:46:22 PDT 2022


fdeazeve created this revision.
fdeazeve added a reviewer: aprantl.
Herald added a project: All.
fdeazeve requested review of this revision.
Herald added a project: LLDB.
Herald added a subscriber: lldb-commits.

The test currently sets `USE_LIBSTDCPP = 0`, which is curious given the
behavior of `and` and `or` in Makefiles (the contents of the variables
are not important). In particular, this causes the tests to not use the
standard libraries appropriately.

To capture the actual intent of the test, we're changing this to
`USE_LIBCXX=1`.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D136171

Files:
  lldb/test/API/functionalities/data-formatter/data-formatter-skip-summary/Makefile


Index: lldb/test/API/functionalities/data-formatter/data-formatter-skip-summary/Makefile
===================================================================
--- lldb/test/API/functionalities/data-formatter/data-formatter-skip-summary/Makefile
+++ lldb/test/API/functionalities/data-formatter/data-formatter-skip-summary/Makefile
@@ -1,7 +1,5 @@
 CXX_SOURCES := main.cpp
-USE_LIBSTDCPP := 0
-
-
+USE_LIBCPP := 1
 
 CXXFLAGS_EXTRAS := -O0
 include Makefile.rules


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D136171.468524.patch
Type: text/x-patch
Size: 461 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20221018/30c371fc/attachment.bin>


More information about the lldb-commits mailing list