[llvm] r260386 - Disable all llvm-config tests for now, will investigate later
Chris Bieneman via llvm-commits
llvm-commits at lists.llvm.org
Mon Dec 12 10:59:27 PST 2016
I know this is an old patch, but I just became aware of it.
This patch is not okay. Disabling a whole test suite without explanation and leaving it disabled for 10 months is very bad.
Ehsan, do you intend to investigate and re-enable these tests?
In r260281 you also deleted a test out of this test suite (also bad), are you going to fix that as well?
> On Feb 10, 2016, at 9:29 AM, Ehsan Akhgari via llvm-commits <llvm-commits at lists.llvm.org> wrote:
> Author: ehsan
> Date: Wed Feb 10 11:29:50 2016
> New Revision: 260386
> URL: http://llvm.org/viewvc/llvm-project?rev=260386&view=rev
> Disable all llvm-config tests for now, will investigate later
> Added: llvm/trunk/test/tools/llvm-config/lit.local.cfg
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/tools/llvm-config/lit.local.cfg?rev=260386&view=auto
> --- llvm/trunk/test/tools/llvm-config/lit.local.cfg (added)
> +++ llvm/trunk/test/tools/llvm-config/lit.local.cfg Wed Feb 10 11:29:50 2016
> @@ -0,0 +1,2 @@
> +# Disable all llvm-config tests for now.
> +config.unsupported = True
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
More information about the llvm-commits