[source maps] Ensure all valid source maps are added instead of failing with the...
authorWalter Erquinigo <waltermelon@fb.com>
Wed, 1 Apr 2020 00:08:40 +0000 (17:08 -0700)
committerWalter Erquinigo <waltermelon@fb.com>
Wed, 1 Apr 2020 20:01:40 +0000 (13:01 -0700)
commit30350c254106ce25b16b13f05e713ceb2b15ce09
treef810779669ab4470b8b0e3f411abb167918f4a99
parente65e677ee4eed83a31503b1a7db3fca56f617eae
[source maps] Ensure all valid source maps are added instead of failing with the first invalid one

Summary:
Several lldb-vscode users have noticed that when a source map rule is invalid (because a folder doesn't exist anymore), the rest of the source maps from their configurations are not applied.
This happens because lldb-vscode executes a single "settings set target.source-map" command with all the source maps and LLDB processes them one by one until one fails.

Instead of doing this, we can process in LLDB all the source map rules and apply the valid ones instead of failing fast.

Reviewers: clayborg, labath, kusmour, aadsm

Subscribers: lldb-commits

Tags: #lldb

Differential Revision: https://reviews.llvm.org/D77186
lldb/source/Interpreter/OptionValuePathMappings.cpp
lldb/test/API/functionalities/source-map/TestTargetSourceMap.py