[Lldb-commits] [PATCH] D60300: [testsuite] Split Objective-C data formatter

Jonas Devlieghere via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Fri Apr 5 11:00:39 PDT 2019


JDevlieghere added a comment.

In D60300#1456485 <https://reviews.llvm.org/D60300#1456485>, @aprantl wrote:

> > What's the motivation behind all the `@no_debug_info_test` decorators? Are those codepaths guaranteed to be tested elsewhere?
>
> As a rule of thumb I'd say, let's only stick `@no_debug_info_test` on tests where a (future) coverage bot proves that it is safe. How does that sound?


Sounds good, I'll remove the decorators and we can do this incrementally.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D60300





More information about the lldb-commits mailing list