[Lldb-commits] [PATCH] D156483: [lldb] Support CTF forward declarations

2023-07-27 Thread Jonas Devlieghere via Phabricator via lldb-commits
JDevlieghere created this revision. JDevlieghere added reviewers: Michael137, bulbazord, jasonmolenda. Herald added a project: All. JDevlieghere requested review of this revision. Add support for CTF forward declarations. https://reviews.llvm.org/D156483 Files: lldb/source/Plugins/SymbolFile/

[Lldb-commits] [PATCH] D156483: [lldb] Support CTF forward declarations

2023-07-27 Thread Alex Langford via Phabricator via lldb-commits
bulbazord added a comment. Looks like a straightforward follow-up to D156447 . One small nit but otherwise looks good. Comment at: lldb/source/Plugins/SymbolFile/CTF/SymbolFileCTF.cpp:530 + CompilerType forward_compiler_type = m_ast->CreateRe

[Lldb-commits] [PATCH] D156483: [lldb] Support CTF forward declarations

2023-07-27 Thread Jonas Devlieghere via Phabricator via lldb-commits
JDevlieghere updated this revision to Diff 544925. JDevlieghere added a comment. Address @bulbazord's feedback. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D156483/new/ https://reviews.llvm.org/D156483 Files: lldb/source/Plugins/SymbolFile/CTF/CTFTypes.h lldb/source/Plugins/Symbol

[Lldb-commits] [PATCH] D156483: [lldb] Support CTF forward declarations

2023-07-28 Thread Jonas Devlieghere via Phabricator via lldb-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG9c70a3d9178f: [lldb] Support CTF forward declarations (authored by JDevlieghere). Herald added a project: LLDB. Changed prior to commit: https://reviews.llvm.org/D156483?vs=544925&id=545324#toc Reposit