[Lldb-commits] [PATCH] D134344: [WIP][lldb][test] 1 - Remove gmodules debug_info variant: add decorator for API tests that explicitly test gmodules

Adrian Prantl via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Wed Sep 21 08:42:37 PDT 2022


aprantl added a comment.

Thanks! As the one who introduced this feature back in the day (and against Pavel's resistance) I've come around to seeing more value in targeted tests than a spray-paint approach of running the entire testsuite. It doesn't catch many interesting issues, since most tests try to avoid external dependencies (that could be built as modules) anyway, and we can write dedicated tests for libcxx and Objective-C modules like Foundation.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D134344/new/

https://reviews.llvm.org/D134344



More information about the lldb-commits mailing list