[Lldb-commits] [PATCH] D57466: [lldb] Relax libc++ ABI version checking

2019-01-30 Thread Tom Anderson via Phabricator via lldb-commits
thomasanderson created this revision. thomasanderson added reviewers: EricWF, thakis. Herald added subscribers: abidh, jfb. libc++ has programmable ABI versioning controllable with the _LIBCPP_ABI_VERSION macro. Currently there are at least 3 settings used in real systems (1 as the default, ndk1

[Lldb-commits] [PATCH] D57466: [lldb] Relax libc++ ABI version checking

2019-01-30 Thread Tom Anderson via Phabricator via lldb-commits
thomasanderson updated this revision to Diff 184418. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57466/new/ https://reviews.llvm.org/D57466 Files: source/Plugins/Language/CPlusPlus/CPlusPlusLanguage.cpp Index: source/Plugins/Language/CPlusPlus/CPlusPlusLanguage.cpp =

[Lldb-commits] [PATCH] D57466: [lldb] Relax libc++ ABI version checking

2019-01-30 Thread Tom Anderson via Phabricator via lldb-commits
thomasanderson added a comment. In D57466#1377696 , @davide wrote: > As long as this doesn't break macOS, it's fine. (just run the tests and watch > the bots). Latest revision fixes 2 test failures on Linux. The first was because ".+" was matching too

[Lldb-commits] [PATCH] D57466: [lldb] Relax libc++ ABI version checking

2019-02-01 Thread Tom Anderson via Phabricator via lldb-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL352899: [lldb] Relax libc++ ABI version checking (authored by thomasanderson, committed by ). Herald added a project: LLVM. Changed prior to commit: https://reviews.llvm.org/D57466?vs=184418&id=184798#t