Page MenuHomePhabricator
Feed Advanced Search

Today

teemperor added a reviewer for D69933: [ASTImporter] Limit imports of structs: teemperor.
Wed, Jan 22, 2:08 AM · Restricted Project, Restricted Project

Yesterday

teemperor committed rGc9a39a896c95: [lldb] Add a display name to ClangASTContext instances (authored by teemperor).
[lldb] Add a display name to ClangASTContext instances
Tue, Jan 21, 11:57 PM
teemperor closed D72391: [lldb] Add a display name to ClangASTContext instances.
Tue, Jan 21, 11:57 PM · Restricted Project
teemperor added a comment to D69933: [ASTImporter] Limit imports of structs.

I really don't think the ASTImporter should ever manipulate records in the source context (effectively the source context should be considered immutable). It also seems *very* wrong that what we import depends in any way on a previous expression so I agree we should fix that. In theory the ImportDefinition call in the ASTImporter shouldn't do any real work as we have the MinimalImport mode on in LLDB so it should only load some bare bone record with external storage IIUC. So I think the original version of the patch seems like a better approach to me from a quick glance.

Tue, Jan 21, 11:55 AM · Restricted Project, Restricted Project
teemperor committed rG1f7b95d038e6: [lldb][NFC] Convert LLDB_LOGF to LLDB_LOG in ClangASTSource.cpp (authored by teemperor).
[lldb][NFC] Convert LLDB_LOGF to LLDB_LOG in ClangASTSource.cpp
Tue, Jan 21, 4:48 AM
teemperor added a comment to D73018: [lldb] Add SystemInitializerAllPlugins and delete copy-pasted Init code in SystemInitializerFull and SystemInitializerTest.

This still leaves the question of the script interpreter plugins, which are suspiciously *not* included in "all plugins". The script interpreters are quite special, so I think it's fine to handle them separately -- the question is just how to convey that distinction. Move them into a different top level folder? Call this SystemInitializerMostPlugins ?

Tue, Jan 21, 3:43 AM · Restricted Project
teemperor added a comment to D73018: [lldb] Add SystemInitializerAllPlugins and delete copy-pasted Init code in SystemInitializerFull and SystemInitializerTest.

I moved the single non-plugin call back to the original Full/Test subclasses so the name is now correct. Also I removed all the duplicated linking flags that I forgot to remove before.

Tue, Jan 21, 1:35 AM · Restricted Project
teemperor added inline comments to D72909: Make SymbolFileDWARF::ParseLineTable use std::sort instead of insertion sort.
Tue, Jan 21, 1:35 AM · Restricted Project
teemperor updated the diff for D73018: [lldb] Add SystemInitializerAllPlugins and delete copy-pasted Init code in SystemInitializerFull and SystemInitializerTest.
  • Removed all the duplicated linking flags too.
Tue, Jan 21, 1:35 AM · Restricted Project

Mon, Jan 20

teemperor updated subscribers of D72557: Add pretty printers for llvm::PointerIntPair and llvm::PointerUnion..
Mon, Jan 20, 1:32 PM · Restricted Project, Restricted Project, Restricted Project, Restricted Project
teemperor committed rG65bab53afb8b: [lldb][NFC] Add test for iterator invalidation during code completion. (authored by teemperor).
[lldb][NFC] Add test for iterator invalidation during code completion.
Mon, Jan 20, 6:08 AM
teemperor updated the summary of D73038: 64 bit SourceLocs.
Mon, Jan 20, 5:49 AM
teemperor created D73038: 64 bit SourceLocs.
Mon, Jan 20, 5:49 AM
teemperor committed rG22447a61d405: [lldb] Mark the implicit copy constructor as deleted when a move constructor is… (authored by teemperor).
[lldb] Mark the implicit copy constructor as deleted when a move constructor is…
Mon, Jan 20, 5:40 AM
teemperor closed D72694: [lldb] Mark the implicit copy constructor as deleted when a move constructor is provided..
Mon, Jan 20, 5:40 AM · Restricted Project
teemperor added a comment to D72694: [lldb] Mark the implicit copy constructor as deleted when a move constructor is provided..

Thanks for the review!

Mon, Jan 20, 5:21 AM · Restricted Project
teemperor updated the diff for D72694: [lldb] Mark the implicit copy constructor as deleted when a move constructor is provided..
  • Added a simple expr evaluation as an additional way to trigger the error.
Mon, Jan 20, 5:21 AM · Restricted Project
teemperor planned changes to D73024: [lldb] Complete return types of CXXMethodDecls to prevent crashing due to covariant return types.

Actually that seems to be more complicated in case we get the method decl by asking the ASTImporter for a definition (which will skip this code).

Mon, Jan 20, 4:35 AM · Restricted Project
teemperor requested changes to D72953: Fix the handling of unnamed bit-fields when parsing DWARF.
Mon, Jan 20, 3:58 AM
teemperor added a comment to D72953: Fix the handling of unnamed bit-fields when parsing DWARF.

(Jim pointed out that we land this without expect_expr to make back porting easier but somehow Phabricator didn't add his comment to the review here. expect_expr is not yet in the downstream Github branch but it is in the 10 release branch, so that makes sense to me).

Mon, Jan 20, 3:58 AM
teemperor created D73024: [lldb] Complete return types of CXXMethodDecls to prevent crashing due to covariant return types.
Mon, Jan 20, 3:21 AM · Restricted Project
teemperor updated the summary of D73018: [lldb] Add SystemInitializerAllPlugins and delete copy-pasted Init code in SystemInitializerFull and SystemInitializerTest.
Mon, Jan 20, 1:02 AM · Restricted Project
teemperor created D73018: [lldb] Add SystemInitializerAllPlugins and delete copy-pasted Init code in SystemInitializerFull and SystemInitializerTest.
Mon, Jan 20, 1:02 AM · Restricted Project

Sun, Jan 19

teemperor added a comment to D73016: [lldb/CMake] Make it possible to disable plugins at configuration time.

How does this deal with linking flags etc.? Your cmake cache disabled LLDB_PLUGIN_SYMBOLFILE_NATIVEPDB_BUILD (well, it would without the LDB typo) but that just gives me a build that fails to link with ld: error: unable to find library -llldbPluginSymbolFileNativePDB.

Sun, Jan 19, 11:48 PM · Restricted Project

Fri, Jan 17

teemperor added a comment to D72953: Fix the handling of unnamed bit-fields when parsing DWARF.

(Just some quick comments, will review this properly during normal working hours)

Fri, Jan 17, 2:24 PM
teemperor removed reviewers for D72946: [lldb] Remove ClangASTImporter reference from Target: martong, a.sidorin.
Fri, Jan 17, 12:57 PM · Restricted Project
teemperor requested changes to D72946: [lldb] Remove ClangASTImporter reference from Target.

I wish we could do this without a global map. Also the ClangASTImporter shouldn't have a dependency on Target (I'm actually surprised this compiles without an additional include).

Fri, Jan 17, 12:57 PM · Restricted Project
teemperor committed rG791f132132b2: [lldb] Remove out of order OperatingSystemPython::Terminate call in… (authored by teemperor).
[lldb] Remove out of order OperatingSystemPython::Terminate call in…
Fri, Jan 17, 4:07 AM
teemperor committed rGf2d41ad0e7e0: [lldb] Add missing terminate calls to Python/Lua subsystems (authored by teemperor).
[lldb] Add missing terminate calls to Python/Lua subsystems
Fri, Jan 17, 3:51 AM
teemperor added inline comments to D72909: Make SymbolFileDWARF::ParseLineTable use std::sort instead of insertion sort.
Fri, Jan 17, 3:48 AM · Restricted Project
teemperor committed rGc3ab790c8f5d: [lldb][NFC] Resynchronize Init/Terminate calls in SystemInitializerFull/Test. (authored by teemperor).
[lldb][NFC] Resynchronize Init/Terminate calls in SystemInitializerFull/Test.
Fri, Jan 17, 2:38 AM
teemperor committed rG6b840834cd50: [lldb][NFC] Delete unused lldb/source/Plugins/LanguageRuntime/Go/CMakeLists.txt (authored by teemperor).
[lldb][NFC] Delete unused lldb/source/Plugins/LanguageRuntime/Go/CMakeLists.txt
Fri, Jan 17, 12:59 AM

Thu, Jan 16

teemperor committed rG5ac610668a7c: [lldb] Re-add NSDate formatter (authored by teemperor).
[lldb] Re-add NSDate formatter
Thu, Jan 16, 11:59 PM
teemperor committed rG81eaa3ddd060: [lldb][NFC] Delete TestDataFormatterObjCNSDate.py (authored by teemperor).
[lldb][NFC] Delete TestDataFormatterObjCNSDate.py
Thu, Jan 16, 8:29 AM
teemperor committed rG39d6b6c21f74: [lldb][NFC] Migrate several tests to expect_expr (authored by teemperor).
[lldb][NFC] Migrate several tests to expect_expr
Thu, Jan 16, 4:47 AM
teemperor added a comment to D72684: [lldb][NFC] Rename ClangASTContext to TypeSystemClang.

Are there any objections against this? Otherwise I'll land this tomorrow.

Thu, Jan 16, 1:31 AM · Restricted Project
teemperor committed rG4f244bba4f66: [lldb] Fix asan failures in data-formatter-objc tests (authored by teemperor).
[lldb] Fix asan failures in data-formatter-objc tests
Thu, Jan 16, 1:03 AM

Wed, Jan 15

teemperor committed rG13f22f5d5958: [lldb] Add expect_expr function for testing expression evaluation in dotests. (authored by teemperor).
[lldb] Add expect_expr function for testing expression evaluation in dotests.
Wed, Jan 15, 4:10 AM
teemperor closed D70314: [lldb] Add expect_expr function for testing expression evaluation in dotests..
Wed, Jan 15, 4:10 AM · Restricted Project
teemperor retitled D70314: [lldb] Add expect_expr function for testing expression evaluation in dotests. from [lldb] Add better test commands for expression evaluation to [lldb] Add expect_expr function for testing expression evaluation in dotests..
Wed, Jan 15, 4:01 AM · Restricted Project
teemperor updated the diff for D70314: [lldb] Add expect_expr function for testing expression evaluation in dotests..
  • Removed substr functionality.
  • Using frame() now.
Wed, Jan 15, 3:15 AM · Restricted Project
teemperor updated the diff for D70314: [lldb] Add expect_expr function for testing expression evaluation in dotests..
  • Removed everything that is not summary or value.
Wed, Jan 15, 2:47 AM · Restricted Project
teemperor updated the diff for D72684: [lldb][NFC] Rename ClangASTContext to TypeSystemClang.
  • Replaced dummy contents with actual patch
Wed, Jan 15, 1:23 AM · Restricted Project
teemperor added a comment to D70314: [lldb] Add expect_expr function for testing expression evaluation in dotests..

I also got rid of the expr->frame var->expr flow and it's not just expr->frame var. I don't want to remove them as we found two formatter bugs by testing both but once these thing don't break constantly then we can remove the 'expr' variant from the simple expression function.

Wed, Jan 15, 1:14 AM · Restricted Project
teemperor updated the diff for D70314: [lldb] Add expect_expr function for testing expression evaluation in dotests..
  • Moved to using the SBAPI.
Wed, Jan 15, 1:05 AM · Restricted Project

Tue, Jan 14

teemperor added a comment to D72684: [lldb][NFC] Rename ClangASTContext to TypeSystemClang.

We might also want to move these into lldb/source/Plugins/TypeSystem as well to complete this refactor?

Tue, Jan 14, 1:28 PM · Restricted Project
teemperor updated the diff for D72694: [lldb] Mark the implicit copy constructor as deleted when a move constructor is provided..
  • Readd test files.
Tue, Jan 14, 8:02 AM · Restricted Project
teemperor updated the diff for D72694: [lldb] Mark the implicit copy constructor as deleted when a move constructor is provided..
  • Rebase.
Tue, Jan 14, 6:16 AM · Restricted Project
teemperor updated the diff for D72684: [lldb][NFC] Rename ClangASTContext to TypeSystemClang.
  • Revert to original dummy patch.
Tue, Jan 14, 6:16 AM · Restricted Project
teemperor added a comment to D72684: [lldb][NFC] Rename ClangASTContext to TypeSystemClang.

Huh, seems I uploaded to the wrong review

Tue, Jan 14, 6:06 AM · Restricted Project
teemperor updated the diff for D72684: [lldb][NFC] Rename ClangASTContext to TypeSystemClang.
  • Rebased to get rid of shady StringRef -> C-String conversion.
Tue, Jan 14, 5:35 AM · Restricted Project
teemperor committed rG3f944a8b8ca8: [lldb][NFC] Make name parameter in AddMethodToCXXRecordType a StringRef (authored by teemperor).
[lldb][NFC] Make name parameter in AddMethodToCXXRecordType a StringRef
Tue, Jan 14, 5:27 AM
teemperor created D72698: [lldb] Add method decls to a CXXRecordDecl only after all their properties are defined.
Tue, Jan 14, 5:26 AM · Restricted Project
teemperor added a comment to D72694: [lldb] Mark the implicit copy constructor as deleted when a move constructor is provided..

FWIW this patch makes me realise that we really should have a Sema around when we create the module AST as this would prevent issues like this and saves us from simulating Sema behavior. That's probably yet another large refactoring so I don't have time for that soon-ish but if anyone wants to do this then be my guest.

Tue, Jan 14, 5:16 AM · Restricted Project
teemperor updated the diff for D72694: [lldb] Mark the implicit copy constructor as deleted when a move constructor is provided..
  • Upload correct revision of the test file.
Tue, Jan 14, 5:16 AM · Restricted Project
teemperor created D72694: [lldb] Mark the implicit copy constructor as deleted when a move constructor is provided..
Tue, Jan 14, 5:09 AM · Restricted Project
teemperor accepted D72510: [lldb/Expression] Improve interpreter error message with a non-running target.

LGTM, thanks!

Tue, Jan 14, 3:50 AM · Restricted Project
teemperor committed rG9492e9d8cfd3: [lldb][NFC] Cleanup ClangASTContext::CompleteTagDeclarationDefinition (authored by teemperor).
[lldb][NFC] Cleanup ClangASTContext::CompleteTagDeclarationDefinition
Tue, Jan 14, 3:05 AM
teemperor added a comment to D72684: [lldb][NFC] Rename ClangASTContext to TypeSystemClang.

Both TypeSystemClang and ClangTypeSystem works for me.

Tue, Jan 14, 2:00 AM · Restricted Project
teemperor committed rGd8ffd601d523: [lldb][NFC] Rewrite python_api/rdar-12481949 test (authored by teemperor).
[lldb][NFC] Rewrite python_api/rdar-12481949 test
Tue, Jan 14, 1:14 AM
teemperor closed D72593: [lldb][NFC] Rewrite python_api/rdar-12481949 test.
Tue, Jan 14, 1:14 AM · Restricted Project
teemperor committed rGf18370fe0e75: [lldb] Don't defend against internal LLVM errors in IRInterpreter (authored by teemperor).
[lldb] Don't defend against internal LLVM errors in IRInterpreter
Tue, Jan 14, 1:04 AM
teemperor closed D72596: [lldb] Don't defend against internal LLVM errors in IRInterpreter.
Tue, Jan 14, 1:04 AM · Restricted Project
teemperor added reviewers for D72684: [lldb][NFC] Rename ClangASTContext to TypeSystemClang: JDevlieghere, davide.
Tue, Jan 14, 12:54 AM · Restricted Project
teemperor added a comment to D72684: [lldb][NFC] Rename ClangASTContext to TypeSystemClang.

We are currently after a fresh rebranch downstream so now is a good time to rename this without causing merge-conflict hell later on. It would be useful if could get consensus if and to what we should rename this class soon-ish so that I can make this move as painless as possible.

Tue, Jan 14, 12:54 AM · Restricted Project
teemperor added a reviewer for D72684: [lldb][NFC] Rename ClangASTContext to TypeSystemClang: labath.
Tue, Jan 14, 12:54 AM · Restricted Project
teemperor created D72684: [lldb][NFC] Rename ClangASTContext to TypeSystemClang.
Tue, Jan 14, 12:46 AM · Restricted Project
teemperor committed rG61b6a4e82653: [lldb] Fix that SBThread.GetStopDescription is returning strings with… (authored by teemperor).
[lldb] Fix that SBThread.GetStopDescription is returning strings with…
Tue, Jan 14, 12:36 AM
teemperor closed D72086: [lldb] Fix that SBThread.GetStopDescription is returning strings with uninitialized memory at the end..
Tue, Jan 14, 12:36 AM · Restricted Project
teemperor added a comment to rG61bd19206f61: [libc++] Explicitly enumerate std::string external instantiations..

FWIW we solved the LLDB problems.

Tue, Jan 14, 12:07 AM

Mon, Jan 13

teemperor added a comment to D72447: [lldb] Mark several tests as not dependent on debug info.

This introduced a failure on the Windows LLDB bot (there were already other failing tests though, so maybe you missed it):

http://lab.llvm.org:8011/builders/lldb-x64-windows-ninja/builds/12554

Mon, Jan 13, 1:24 PM · Restricted Project
teemperor committed rG577efd09e3b7: [lldb] Revert ddf044290ede for TestProcessAPI.py (authored by teemperor).
[lldb] Revert ddf044290ede for TestProcessAPI.py
Mon, Jan 13, 1:15 PM
teemperor added a reverting change for rGddf044290ede: [lldb] Mark several tests as not dependent on debug info: rG577efd09e3b7: [lldb] Revert ddf044290ede for TestProcessAPI.py.
Mon, Jan 13, 1:15 PM
teemperor committed rGc1fbede984ec: [lldb][NFC] Remove debug print statement from TestExprDiagnostics.py (authored by teemperor).
[lldb][NFC] Remove debug print statement from TestExprDiagnostics.py
Mon, Jan 13, 5:20 AM
teemperor updated the summary of D72596: [lldb] Don't defend against internal LLVM errors in IRInterpreter.
Mon, Jan 13, 4:15 AM · Restricted Project
teemperor updated the summary of D72596: [lldb] Don't defend against internal LLVM errors in IRInterpreter.
Mon, Jan 13, 3:20 AM · Restricted Project
teemperor updated the summary of D72596: [lldb] Don't defend against internal LLVM errors in IRInterpreter.
Mon, Jan 13, 3:19 AM · Restricted Project
teemperor created D72596: [lldb] Don't defend against internal LLVM errors in IRInterpreter.
Mon, Jan 13, 3:19 AM · Restricted Project
teemperor committed rGaf4adb07cd18: [lldb][NFC] Use range-based for loops in IRInterpreter (authored by teemperor).
[lldb][NFC] Use range-based for loops in IRInterpreter
Mon, Jan 13, 3:00 AM
teemperor created D72593: [lldb][NFC] Rewrite python_api/rdar-12481949 test.
Mon, Jan 13, 2:59 AM · Restricted Project
teemperor committed rGddf044290ede: [lldb] Mark several tests as not dependent on debug info (authored by teemperor).
[lldb] Mark several tests as not dependent on debug info
Mon, Jan 13, 1:43 AM
teemperor closed D72447: [lldb] Mark several tests as not dependent on debug info.
Mon, Jan 13, 1:43 AM · Restricted Project
teemperor added a comment to D72086: [lldb] Fix that SBThread.GetStopDescription is returning strings with uninitialized memory at the end..

I made a new typemap now that is only for GetStopDescription and it's snprintf semantics. This keeps all the other functions working. I kept the old behavior of requiring a >0 buffer size etc. in tact for now.

Mon, Jan 13, 1:02 AM · Restricted Project
teemperor updated the summary of D72086: [lldb] Fix that SBThread.GetStopDescription is returning strings with uninitialized memory at the end..
Mon, Jan 13, 1:02 AM · Restricted Project
teemperor requested review of D72086: [lldb] Fix that SBThread.GetStopDescription is returning strings with uninitialized memory at the end..
Mon, Jan 13, 1:02 AM · Restricted Project
teemperor updated the diff for D72086: [lldb] Fix that SBThread.GetStopDescription is returning strings with uninitialized memory at the end..
  • Move to use a new typemap for GetStopDescription.
Mon, Jan 13, 12:55 AM · Restricted Project
teemperor requested changes to D72510: [lldb/Expression] Improve interpreter error message with a non-running target.

To give this more context: The IRInterpreter can interpret function calls but only on Hexagon (see D9404). The reason why we essentially always see this error message when the target isn't running is because most people don't target Hexagon and function calls is the most common reason IRInterpreter::CanInterpret returns an error. However, it *could* also fail for any of the other error messages in this function.

Mon, Jan 13, 12:25 AM · Restricted Project

Fri, Jan 10

teemperor committed rG9e13cff44d6b: [lldb] Fix TestClangASTContext.TestFunctionTemplateInRecordConstruction in… (authored by teemperor).
[lldb] Fix TestClangASTContext.TestFunctionTemplateInRecordConstruction in…
Fri, Jan 10, 12:35 PM
teemperor closed D72359: [lldb] Fix TestClangASTContext.TestFunctionTemplateInRecordConstruction in Debug builds.
Fri, Jan 10, 12:35 PM · Restricted Project
teemperor committed rG77da826edad0: [lldb] Make CompleteTagDeclsScope completion order deterministic (authored by teemperor).
[lldb] Make CompleteTagDeclsScope completion order deterministic
Fri, Jan 10, 12:24 PM
teemperor closed D72495: [lldb] Make CompleteTagDeclsScope completion order deterministic.
Fri, Jan 10, 12:24 PM · Restricted Project
teemperor committed rGef239972614c: [lldb] Remove FieldDecl stealing hack by rerouting indirect imports to the… (authored by teemperor).
[lldb] Remove FieldDecl stealing hack by rerouting indirect imports to the…
Fri, Jan 10, 10:22 AM
teemperor closed D72507: [lldb] Remove FieldDecl stealing hack by rerouting indirect imports to the original AST.
Fri, Jan 10, 10:22 AM · Restricted Project
teemperor added a comment to D72507: [lldb] Remove FieldDecl stealing hack by rerouting indirect imports to the original AST.

FWIW, this isn't really a finished patch yet as I actually wanted to write several unit tests for this, but as it fixes the gmodules failures on master I thought I might as well put it up now.

Fri, Jan 10, 7:15 AM · Restricted Project
teemperor created D72507: [lldb] Remove FieldDecl stealing hack by rerouting indirect imports to the original AST.
Fri, Jan 10, 7:15 AM · Restricted Project
teemperor committed rG902974277d50: Data formatters: Look through array element typedefs (authored by jarin).
Data formatters: Look through array element typedefs
Fri, Jan 10, 2:47 AM
teemperor closed D72133: Data formatters: Look through array element typedefs.
Fri, Jan 10, 2:47 AM · Restricted Project
teemperor added a comment to D72133: Data formatters: Look through array element typedefs.

Actually the failures are gmodules only so I'll land this now.

Fri, Jan 10, 2:11 AM · Restricted Project
teemperor added a comment to D72133: Data formatters: Look through array element typedefs.

Raphael, could you possibly land the patch for me? Thanks!

Fri, Jan 10, 2:11 AM · Restricted Project
teemperor updated the summary of D72495: [lldb] Make CompleteTagDeclsScope completion order deterministic.
Fri, Jan 10, 2:11 AM · Restricted Project