[Lldb-commits] [PATCH] D47708: PDB support of function-level linking and splitted functions
Leonard Mosescu via lldb-commits
lldb-commits at lists.llvm.org
Fri Jun 8 10:01:54 PDT 2018
Doesn't the LIT based test drop the split-function case (originally
produced with PGO)?
Sorry for being late to the party, but it seems beneficial to have both LIT
*and* checked in binaries since in general they are complementary: checking
against freshly built binaries only covers a matching set of toolchain
components (in particular it's hard to cover the cross-targeting scenarios).
Other than the inconvenience with Phabricator, is there a reason not to
include the original tests as well? The size of the binaries?
On Fri, Jun 8, 2018 at 1:27 AM, Pavel Labath via Phabricator via
lldb-commits <lldb-commits at lists.llvm.org> wrote:
> labath added a comment.
> Thank you for your patience. I am very happy with the final result here.
> I personally like to keep number of files in a test minimal (hence I
> inlined the test into the cpp file), but overall that's not really
> lldb-commits mailing list
> lldb-commits at lists.llvm.org
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the lldb-commits