Skip to content

Commit df15e8d

Browse files
royitaqiGeorge Hu
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 ff43800 commit df15e8d

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
@@ -1248,6 +1248,11 @@ static void FilterAndGetValueForKey(const lldb::SBStructuredData data,
12481248
void addStatisticsSummary(lldb::SBTarget &target, llvm::json::Object &event) {
12491249
lldb::SBStatisticsOptions options;
12501250
options.SetSummaryOnly(true);
1251+
// META BEGIN
1252+
// Temporary fix until #97004 is upstreamed.
1253+
// TODO: Revert/delete this block once #97004 is in toolchain/llvm-sand/main
1254+
options.SetIncludeTargets(true);
1255+
// META END
12511256
lldb::SBStructuredData statistics = target.GetStatistics(options);
12521257

12531258
bool is_dictionary =

0 commit comments

Comments
 (0)