FailedChanges

Summary

  1. [lldb-test] Modify lldb-test to print out ASTs from symbol file (details)
  2. Update clang module map for new excluded .def file. (details)
  3. [lldb] Add import-std-module/sysroot to the libc++ test category. (details)
  4. [clang][IFS] Fixing assert in clang interface stubs for enums, records, (details)
  5. [lldb] Mark import-std-module/empty-module as libc++ test (details)
  6. gn build: (manually) merge r374110 (details)
  7. IOHandler: fall back on File::Read if a FILE* isn't available. (details)
  8. [lldb] Fix python packages install path (details)
Commit 5f46982b452b685419629b23d530281df2298536 by Shafik Yaghmour
[lldb-test] Modify lldb-test to print out ASTs from symbol file
Summary: Currently when invoking lldb-test symbols -dump-ast it parses
all the debug symbols and calls print(...) on the TranslationUnitDecl.
While useful the TranslationUnitDecl::print(...) method gives us a
higher level view then the dump from ASTDumper which is what we get when
we invoke dump() on a specific AST node. The main motivation for this
change is allow us to verify that the AST nodes we create when we parse
DWARF. For example in order to verify we are correctly using
DIFlagExportSymbols added by D66667
Differential Revision: https://reviews.llvm.org/D67994
llvm-svn: 374570
The file was modifiedlldb/include/lldb/Symbol/ClangASTContext.h (diff)
The file was modifiedlldb/tools/lldb-test/lldb-test.cpp (diff)
The file was modifiedlldb/source/Plugins/SymbolFile/DWARF/SymbolFileDWARF.cpp (diff)
The file was modifiedlldb/source/Symbol/ClangASTContext.cpp (diff)
Commit 1edb7e0b6f390b066f5218208a7c8ac974ee243c by richard-llvm
Update clang module map for new excluded .def file.
llvm-svn: 374571
The file was modifiedclang/include/clang/module.modulemap (diff)
Commit 81018c85b9c8275cc3bd0b60aa7a3ef4b601fa16 by Raphael Isemann
[lldb] Add import-std-module/sysroot to the libc++ test category.
We essentially test libc++ in a sysroot here so let's make sure that we
actually only run this test on platforms where libc++ testing is
enabled.
llvm-svn: 374572
The file was modifiedlldb/packages/Python/lldbsuite/test/commands/expression/import-std-module/sysroot/TestStdModuleSysroot.py (diff)
Commit e3388c42f39b6715e9aa1c7594fb0dcf9429e111 by puyan
[clang][IFS] Fixing assert in clang interface stubs for enums, records,
typedefs
The clang IFS ASTConsumer was asserting on enums, records (struct
definitions in C), and typedefs. All it needs to do is skip them because
the stub just needs to expose global object instances and functions.
Differential Revision: https://reviews.llvm.org/D68859
llvm-svn: 374573
The file was modifiedclang/lib/Frontend/InterfaceStubFunctionsConsumer.cpp (diff)
The file was addedclang/test/InterfaceStubs/noninstancetypes.c
Commit 766a0cb1e082efa85f01187b68c9a784b3ed62f5 by Raphael Isemann
[lldb] Mark import-std-module/empty-module as libc++ test
Same as with the import-std-module/sysroot test before.
llvm-svn: 374574
The file was modifiedlldb/packages/Python/lldbsuite/test/commands/expression/import-std-module/empty-module/TestEmptyStdModule.py (diff)
Commit e695d3c695652d4cc4afcb52112d04f3b25fe851 by nicolasweber
gn build: (manually) merge r374110
llvm-svn: 374575
The file was modifiedllvm/utils/gn/secondary/clang/test/BUILD.gn (diff)
Commit b3faa01ff9625cc00d98d5fffd597c64d494c349 by lawrence_danna
IOHandler: fall back on File::Read if a FILE* isn't available.
Summary: IOHandler needs to read lines of input from a lldb::File. The
way it currently does this using, FILE*, which is something we want to
avoid now.   I'd prefer to just replace the FILE* code with calls to
File::Read, but it contains an awkward and delicate workaround specific
to ctrl-C handling on windows, and it's not clear if or how that
workaround would translate to lldb::File.
So in this patch, we use use the FILE* if it's available, and only fall
back on File::Read if that's the only option.
I think this is a reasonable approach here for two reasons.  First is
that interactive terminal support is the one area where FILE* can't be
avoided.   We need them for libedit and curses anyway, and using them
here as well is consistent with that pattern.
The second reason is that the comments express a hope that the
underlying windows bug that's being worked around will be fixed one day,
so hopefully when that happens, that whole path can be deleted.
Reviewers: JDevlieghere, jasonmolenda, labath, lanza
Reviewed By: labath
Subscribers: lldb-commits
Tags: #lldb
Differential Revision: https://reviews.llvm.org/D68622
llvm-svn: 374576
The file was modifiedlldb/include/lldb/Core/IOHandler.h (diff)
The file was modifiedlldb/packages/Python/lldbsuite/test/python_api/file_handle/TestFileHandle.py (diff)
The file was modifiedlldb/source/Core/IOHandler.cpp (diff)
Commit 6aacd968754341a0743a427d842cacb91637279c by hhb
[lldb] Fix python packages install path
Reviewers: mgorny
Subscribers: lldb-commits
Tags: #lldb
Differential Revision: https://reviews.llvm.org/D68878
llvm-svn: 374577
The file was modifiedlldb/CMakeLists.txt (diff)