[Lldb-commits] [PATCH] D60817: [NativePDB] Add anonymous namespaces support

2019-04-19 Thread Adrian McCarthy via Phabricator via lldb-commits
amccarth accepted this revision.
amccarth added a comment.
This revision is now accepted and ready to land.

Thanks for the changes.  If the tests pass, then this LGTM.  Just check the one 
last question I added about the `AC1D` test.

(The diff in Phabricator looks very confused, as though you deleted several 
hundred lines.  Perhaps because the original diff had context and this one 
didn't.  If I'm interpretting the diff correctly, this looks good now.)




Comment at: lit/SymbolFile/NativePDB/ast-types.cpp:22


Should this one start with FIXME rather than CHECK?  Isn't this the case that's 
currently broken?


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

https://reviews.llvm.org/D60817



___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits


[Lldb-commits] [PATCH] D60588: Adjusting libc++ std::list formatter to act better with pointers and references and adding a test to cover a previous related fix

2019-04-19 Thread Jim Ingham via Phabricator via lldb-commits
jingham accepted this revision.
jingham added a comment.
This revision is now accepted and ready to land.

LGTM


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

https://reviews.llvm.org/D60588



___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits


[Lldb-commits] [lldb] r358773 - [Docs] Make Doxygen functional

2019-04-19 Thread Jonas Devlieghere via lldb-commits
Author: jdevlieghere
Date: Fri Apr 19 11:02:35 2019
New Revision: 358773

URL: http://llvm.org/viewvc/llvm-project?rev=358773&view=rev
Log:
[Docs] Make Doxygen functional

This fixes the doxygen configuration to be functional again. I removed
the customer header and footer, as well as the no-longer-existent style
sheet. I also widened the scope of the documentation, from just the
public API to include the private interfaces as well.

Added:
lldb/trunk/docs/doxygen-mainpage.dox
Removed:
lldb/trunk/docs/doxygen.footer
lldb/trunk/docs/doxygen.header
lldb/trunk/docs/doxygen.intro
Modified:
lldb/trunk/docs/doxygen.cfg.in

Added: lldb/trunk/docs/doxygen-mainpage.dox
URL: 
http://llvm.org/viewvc/llvm-project/lldb/trunk/docs/doxygen-mainpage.dox?rev=358773&view=auto
==
--- lldb/trunk/docs/doxygen-mainpage.dox (added)
+++ lldb/trunk/docs/doxygen-mainpage.dox Fri Apr 19 11:02:35 2019
@@ -0,0 +1,15 @@
+/// @mainpage LLDB
+///
+/// @section main_intro Introduction
+/// Welcome to LLDB.
+///
+/// This documentation describes both the private and public interface of LLDB.
+/// There are no instructions here on how to use LLDB, only the APIs
+/// that make up the software. For usage instructions, please see
+/// the help command.
+///
+/// @section main_caveat Caveat
+/// This documentation is generated directly from the source code with doxygen.
+/// Since LLDB is constantly under active development, what you're about to
+/// read is out of date! However, it may still be useful since certain portions
+/// of LLDB are very stable.

Modified: lldb/trunk/docs/doxygen.cfg.in
URL: 
http://llvm.org/viewvc/llvm-project/lldb/trunk/docs/doxygen.cfg.in?rev=358773&r1=358772&r2=358773&view=diff
==
--- lldb/trunk/docs/doxygen.cfg.in (original)
+++ lldb/trunk/docs/doxygen.cfg.in Fri Apr 19 11:02:35 2019
@@ -25,7 +25,7 @@ DOXYFILE_ENCODING  = UTF-8
 # The PROJECT_NAME tag is a single word (or a sequence of words surrounded
 # by quotes) that should identify the project.
 
-PROJECT_NAME   = LLVM
+PROJECT_NAME   = LLDB
 
 # The PROJECT_NUMBER tag can be used to enter a project or revision number.
 # This could be handy for archiving the generated documentation or
@@ -581,8 +581,9 @@ WARN_LOGFILE   =
 # directories like "/usr/src/myproject". Separate the files or directories
 # with spaces.
 
-INPUT  = @abs_top_srcdir@/include/lldb/API \
- @abs_top_srcdir@/docs/doxygen.intro
+INPUT  = @abs_top_srcdir@/include/ \
+ @abs_top_srcdir@/source/ \
+ @abs_top_srcdir@/docs/doxygen-mainpage.dox
 
 # This tag can be used to specify the character encoding of the source files
 # that doxygen parses. Internally doxygen uses the UTF-8 encoding, which is
@@ -793,13 +794,13 @@ HTML_FILE_EXTENSION= .html
 # each generated HTML page. If it is left blank doxygen will generate a
 # standard header.
 
-HTML_HEADER= @abs_top_srcdir@/docs/doxygen.header
+HTML_HEADER=
 
 # The HTML_FOOTER tag can be used to specify a personal HTML footer for
 # each generated HTML page. If it is left blank doxygen will generate a
 # standard footer.
 
-HTML_FOOTER= @abs_top_srcdir@/docs/doxygen.footer
+HTML_FOOTER=
 
 # The HTML_STYLESHEET tag can be used to specify a user-defined cascading
 # style sheet that is used by each HTML page. It can be used to
@@ -808,7 +809,7 @@ HTML_FOOTER= @abs_top_srcdir
 # the style sheet file to the HTML output directory, so don't put your own
 # stylesheet in the HTML output directory as well, or it will be erased!
 
-HTML_STYLESHEET= @abs_top_srcdir@/../../docs/doxygen.css
+HTML_STYLESHEET=
 
 # The HTML_COLORSTYLE_HUE tag controls the color of the HTML output.
 # Doxygen will adjust the colors in the stylesheet and background images

Removed: lldb/trunk/docs/doxygen.footer
URL: 
http://llvm.org/viewvc/llvm-project/lldb/trunk/docs/doxygen.footer?rev=358772&view=auto
==
--- lldb/trunk/docs/doxygen.footer (original)
+++ lldb/trunk/docs/doxygen.footer (removed)
@@ -1,13 +0,0 @@
-
-
-Generated on $datetime for http://lldb.llvm.org/";>$projectname by
-http://www.doxygen.org";>$doxygenversion
-Copyright © 2003-2013 University of Illinois at Urbana-Champaign.
-All Rights Reserved.
-
-
-
-
-
-

Removed: lldb/trunk/docs/doxygen.header
URL: 
http://llvm.org/viewvc/llvm-project/lldb/trunk/docs/doxygen.header?rev=358772&view=auto
==
--- lldb/trunk/docs/doxygen.header (original)
+++ lldb/trunk/docs/doxygen.header (removed)
@@ -1,9 +0,0 @@
-
-
-
-
-
-LLVM: $title
-
-
-LLDB API Documentation

Removed: lldb/trunk/docs/doxygen.intro
U

[Lldb-commits] [lldb] r358776 - This test doesn't need to be run for all debug formats.

2019-04-19 Thread Jim Ingham via lldb-commits
Author: jingham
Date: Fri Apr 19 11:46:56 2019
New Revision: 358776

URL: http://llvm.org/viewvc/llvm-project?rev=358776&view=rev
Log:
This test doesn't need to be run for all debug formats.

Modified:

lldb/trunk/packages/Python/lldbsuite/test/functionalities/plugins/python_os_plugin/TestPythonOSPlugin.py

Modified: 
lldb/trunk/packages/Python/lldbsuite/test/functionalities/plugins/python_os_plugin/TestPythonOSPlugin.py
URL: 
http://llvm.org/viewvc/llvm-project/lldb/trunk/packages/Python/lldbsuite/test/functionalities/plugins/python_os_plugin/TestPythonOSPlugin.py?rev=358776&r1=358775&r2=358776&view=diff
==
--- 
lldb/trunk/packages/Python/lldbsuite/test/functionalities/plugins/python_os_plugin/TestPythonOSPlugin.py
 (original)
+++ 
lldb/trunk/packages/Python/lldbsuite/test/functionalities/plugins/python_os_plugin/TestPythonOSPlugin.py
 Fri Apr 19 11:46:56 2019
@@ -16,6 +16,7 @@ import lldbsuite.test.lldbutil as lldbut
 class PluginPythonOSPlugin(TestBase):
 
 mydir = TestBase.compute_mydir(__file__)
+NO_DEBUG_INFO_TESTCASE = True
 
 def test_python_os_plugin(self):
 """Test that the Python operating system plugin works correctly"""


___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits


[Lldb-commits] [lldb] r358777 - [Docs] Add more info about building the docs

2019-04-19 Thread Jonas Devlieghere via lldb-commits
Author: jdevlieghere
Date: Fri Apr 19 12:03:58 2019
New Revision: 358777

URL: http://llvm.org/viewvc/llvm-project?rev=358777&view=rev
Log:
[Docs] Add more info about building the docs

Including the C++ and Python reference.

Modified:
lldb/trunk/docs/resources/build.rst

Modified: lldb/trunk/docs/resources/build.rst
URL: 
http://llvm.org/viewvc/llvm-project/lldb/trunk/docs/resources/build.rst?rev=358777&r1=358776&r2=358777&view=diff
==
--- lldb/trunk/docs/resources/build.rst (original)
+++ lldb/trunk/docs/resources/build.rst Fri Apr 19 12:03:58 2019
@@ -232,21 +232,6 @@ On NetBSD one might run:
   > pkgin install swig python27 cmake ninja-build
 
 
-If you wish to build the optional reference documentation, additional 
dependencies are required:
-
-* Graphviz (for the 'dot' tool).
-* doxygen (only if you wish to build the C++ API reference)
-* epydoc (only if you wish to build the Python API reference)
-
-
-To install the prerequisites for building the documentation (on Debian/Ubuntu) 
do:
-
-::
-
-  > sudo apt-get install doxygen graphviz
-  > sudo pip install epydoc # or install package python-epydoc
-
-
 **Building LLDB**
 
 We first need to checkout the source trees into the appropriate locations. Both
@@ -514,3 +499,30 @@ arm64 build:
 
 Note that currently only lldb-server is functional on android. The lldb client
 is not supported and unlikely to work.
+
+Building The Documentation
+--
+
+If you wish to build the optional (reference) documentation, additional
+dependencies are required:
+
+* Sphinx (for the website)
+* Graphviz (for the 'dot' tool)
+* doxygen (if you wish to build the C++ API reference)
+* epydoc (if you wish to build the Python API reference)
+
+To install the prerequisites for building the documentation (on Debian/Ubuntu)
+do:
+
+::
+
+  > sudo apt-get install doxygen graphviz python3-sphinx
+  > sudo pip install epydoc
+
+To build the documentation, build the desired target(s).
+
+::
+
+  > cmake --build . --target docs-lldb-html
+  > cmake --build . --target lldb-cpp-doc
+  > cmake --build . --target lldb-python-doc


___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits


[Lldb-commits] [lldb] r358792 - [Tests] Split float test into float and doubles

2019-04-19 Thread Jonas Devlieghere via lldb-commits
Author: jdevlieghere
Date: Fri Apr 19 15:37:55 2019
New Revision: 358792

URL: http://llvm.org/viewvc/llvm-project?rev=358792&view=rev
Log:
[Tests] Split float test into float and doubles

As I was waiting for the test suite to complete at 99% I noticed this
test taking quite a bit of time. Since it's easy to split I just went
ahead and did so.

Added:
lldb/trunk/packages/Python/lldbsuite/test/types/TestDoubleTypes.py
lldb/trunk/packages/Python/lldbsuite/test/types/TestDoubleTypesExpr.py
Modified:
lldb/trunk/packages/Python/lldbsuite/test/types/TestFloatTypes.py
lldb/trunk/packages/Python/lldbsuite/test/types/TestFloatTypesExpr.py

Added: lldb/trunk/packages/Python/lldbsuite/test/types/TestDoubleTypes.py
URL: 
http://llvm.org/viewvc/llvm-project/lldb/trunk/packages/Python/lldbsuite/test/types/TestDoubleTypes.py?rev=358792&view=auto
==
--- lldb/trunk/packages/Python/lldbsuite/test/types/TestDoubleTypes.py (added)
+++ lldb/trunk/packages/Python/lldbsuite/test/types/TestDoubleTypes.py Fri Apr 
19 15:37:55 2019
@@ -0,0 +1,36 @@
+"""
+Test that variables of floating point types are displayed correctly.
+"""
+
+from __future__ import print_function
+
+
+import AbstractBase
+import sys
+
+import lldb
+from lldbsuite.test.decorators import *
+from lldbsuite.test.lldbtest import *
+from lldbsuite.test import lldbutil
+
+
+class DoubleTypesTestCase(AbstractBase.GenericTester):
+
+mydir = AbstractBase.GenericTester.compute_mydir(__file__)
+
+def setUp(self):
+# Call super's setUp().
+AbstractBase.GenericTester.setUp(self)
+# disable "There is a running process, kill it and restart?" prompt
+self.runCmd("settings set auto-confirm true")
+self.addTearDownHook(
+lambda: self.runCmd("settings clear auto-confirm"))
+
+def test_double_type(self):
+"""Test that double-type variables are displayed correctly."""
+self.build_and_run('double.cpp', set(['double']))
+
+@skipUnlessDarwin
+def test_double_type_from_block(self):
+"""Test that double-type variables are displayed correctly from a 
block."""
+self.build_and_run('double.cpp', set(['double']), bc=True)

Added: lldb/trunk/packages/Python/lldbsuite/test/types/TestDoubleTypesExpr.py
URL: 
http://llvm.org/viewvc/llvm-project/lldb/trunk/packages/Python/lldbsuite/test/types/TestDoubleTypesExpr.py?rev=358792&view=auto
==
--- lldb/trunk/packages/Python/lldbsuite/test/types/TestDoubleTypesExpr.py 
(added)
+++ lldb/trunk/packages/Python/lldbsuite/test/types/TestDoubleTypesExpr.py Fri 
Apr 19 15:37:55 2019
@@ -0,0 +1,40 @@
+"""
+Test that variable expressions of floating point types are evaluated correctly.
+"""
+
+from __future__ import print_function
+
+
+import AbstractBase
+import sys
+
+import lldb
+from lldbsuite.test.decorators import *
+from lldbsuite.test.lldbtest import *
+from lldbsuite.test import lldbutil
+
+
+class DoubleTypesExprTestCase(AbstractBase.GenericTester):
+
+mydir = AbstractBase.GenericTester.compute_mydir(__file__)
+
+# rdar://problem/8493023
+# test/types failures for Test*TypesExpr.py: element offset computed wrong
+# and sign error?
+
+def setUp(self):
+# Call super's setUp().
+AbstractBase.GenericTester.setUp(self)
+# disable "There is a running process, kill it and restart?" prompt
+self.runCmd("settings set auto-confirm true")
+self.addTearDownHook(
+lambda: self.runCmd("settings clear auto-confirm"))
+
+def test_double_type(self):
+"""Test that double-type variable expressions are evaluated 
correctly."""
+self.build_and_run_expr('double.cpp', set(['double']))
+
+@skipUnlessDarwin
+def test_double_type_from_block(self):
+"""Test that double-type variables are displayed correctly from a 
block."""
+self.build_and_run_expr('double.cpp', set(['double']), bc=True)

Modified: lldb/trunk/packages/Python/lldbsuite/test/types/TestFloatTypes.py
URL: 
http://llvm.org/viewvc/llvm-project/lldb/trunk/packages/Python/lldbsuite/test/types/TestFloatTypes.py?rev=358792&r1=358791&r2=358792&view=diff
==
--- lldb/trunk/packages/Python/lldbsuite/test/types/TestFloatTypes.py (original)
+++ lldb/trunk/packages/Python/lldbsuite/test/types/TestFloatTypes.py Fri Apr 
19 15:37:55 2019
@@ -34,12 +34,3 @@ class FloatTypesTestCase(AbstractBase.Ge
 def test_float_type_from_block(self):
 """Test that float-type variables are displayed correctly from a 
block."""
 self.build_and_run('float.cpp', set(['float']), bc=True)
-
-def test_double_type(self):
-"""Test that double-type variables are displayed correctly."""
-self.build_and_run('double.cpp', set(['double']))
-
-@skipUnlessDarwi