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

Michael Buch via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Fri Sep 23 02:51:21 PDT 2022


Michael137 added a comment.

> (B) Keep the `gmodules` category in the debug_info categories but add an indicator (e.g., by making the `debug_info_categories` a dictionary) that will skip replication if set. That would solve (1). And (2) would work as it does today without changes.

Uploaded alternative diff that implements this option. Seems simpler since tests in the `gmodules` category Just Work and we don't need to special-case `gmodules` in several places

https://reviews.llvm.org/D134524


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