Skip to content

Commit 4eb0196

Browse files
royitaqikusmour
authored andcommitted
Temporarily fix a test: TestVSCode
Summary: This is to temporarily fix the test `TestVSCode`, before [PR llvm#97004](llvm#97004) can be upstreamed and pulled into `toolchain/llvm-sand/main`. The bug was that a previous PR ([llvm#95075](llvm#95075)) accidentally removed the `targets` section from `statistics dump --summary`, which this test relies on. The fix in [PR llvm#97004](llvm#97004) reverts that behavior, while the temporary fix in this diff just explicitly adds an option to make sure that the `targets` section is generated. This diff *can (but doesn't have to)* be reverted after [PR llvm#97004](llvm#97004) has been merged and pulled into `llvm-sand`. Test Plan: Manually tested after applying this diff on top of the release candidate that I'm working on. Reviewers: jeffreytan, #lldb_team Subscribers: gclayton, #lldb_team Differential Revision: https://phabricator.intern.facebook.com/D59139567
1 parent 8dbe2fc commit 4eb0196

File tree

1 file changed

+5
-0
lines changed

1 file changed

+5
-0
lines changed

lldb/tools/lldb-dap/JSONUtils.cpp

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1489,6 +1489,11 @@ void FilterAndGetValueForKey(const lldb::SBStructuredData data, const char *key,
14891489
void addStatisticsSummary(llvm::json::Object &event) {
14901490
lldb::SBStatisticsOptions options;
14911491
options.SetSummaryOnly(true);
1492+
// META BEGIN
1493+
// Temporary fix until #97004 is upstreamed.
1494+
// TODO: Revert/delete this block once #97004 is in toolchain/llvm-sand/main
1495+
options.SetIncludeTargets(true);
1496+
// META END
14921497
lldb::SBStructuredData statistics =
14931498
g_dap.target.GetStatistics(options);
14941499

0 commit comments

Comments
 (0)