[llvm-commits] [polly] r153739 - /polly/trunk/test/Makefile

[Sebastian Pop]

On Fri, Mar 30, 2012 at 4:27 AM, Hongbin Zheng <etherzhhb at gmail.com> wrote:
> Author: ether
> Date: Fri Mar 30 04:27:16 2012
> New Revision: 153739
>
> URL: http://llvm.org/viewvc/llvm-project?rev=153739&view=rev
> Log:
> Make the "all" target depend on polly-test, so that users can run regression
>  tests by simply typing "make -C tools/polly/test", like llvm's regression
>  tests.
>
> Modified:
>    polly/trunk/test/Makefile
>
> Modified: polly/trunk/test/Makefile
> URL: http://llvm.org/viewvc/llvm-project/polly/trunk/test/Makefile?rev=153739&r1=153738&r2=153739&view=diff
> ==============================================================================
> --- polly/trunk/test/Makefile (original)
> +++ polly/trunk/test/Makefile Fri Mar 30 04:27:16 2012
> @@ -31,6 +31,7 @@
>   LIT_ARGS += "--vg"
>  endif
>
> +all:: polly-test

Several of my automatic tests have failed because of this change: when
make has finished building the polly/lib, it automatically goes to
make all in the test dir, and because of this change, it starts
testing polly, even though the rest of llvm and clang have not
finished to be built.

Please fix this by not including the test dir in the polly top level
Makefile DIRS variable:

diff --git a/Makefile b/Makefile
index 2ad5b36..665c13a 100644
--- a/Makefile
+++ b/Makefile
@@ -8,7 +8,7 @@
 # Indicates our relative path to the top of the project's root directory.
 #
 LEVEL = .
-DIRS = lib test tools
+DIRS = lib tools
 EXTRA_DIST = include

 #

Thanks,
Sebastian
--
Qualcomm Innovation Center, Inc is a member of Code Aurora Forum