[libcxx-commits] [PATCH] D156242: [libc++] Removes a broken patch.
Mark de Wever via Phabricator via libcxx-commits
libcxx-commits at lists.llvm.org
Tue Jul 25 10:20:51 PDT 2023
Mordante added a comment.
In D156242#4532561 <https://reviews.llvm.org/D156242#4532561>, @philnik wrote:
> Is this the only test that checks the gdb pretty printer? If yes, then I'm not really comfortable removing it, since we have the gdb pretty printer script too. I realize that it's not tested right now, but having a failing test as a start to fix things is better than having no test at all.
I somewhat agree, but a test that we just disable after we update the LLVM version is not a help either. We haven't tested it for at least clang-15. Having an unused test only gives a false illusion. I added Louis as reviewer since I want to hear his opinion.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D156242/new/
https://reviews.llvm.org/D156242
More information about the libcxx-commits
mailing list