[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