[llvm-commits] [llvm] r81719 - in /llvm/trunk: unittests/Makefile unittests/TestMain.cpp utils/unittest/Makefile utils/unittest/UnitTestMain/ utils/unittest/UnitTestMain/Makefile utils/unittest/UnitTestMain/TestMain.cpp

[Julien Lerouge]

On Sat, Sep 19, 2009 at 11:17:40PM -0700, Daniel Dunbar wrote:
> Ok, this was a preexisting problem with the unit test build but it
> would be nice to fix. I don't think your patch is correct though, it
> changes the flags in the utils/unittest but this isn't how recursive
> make works, the builds in the subdirs just ignore those options which
> means the build gets new warnings. I'll just duplicated the logic into
> UnitTestMain for now, although it probably would be better to suck it
> into the top level Makefile. r82373.
> 
>  - Daniel

Right, sorry for the bad patch, and thanks for the fix!

Julien

-- 
Julien Lerouge
PGP Key Id: 0xB1964A62
PGP Fingerprint: 392D 4BAD DB8B CE7F 4E5F FA3C 62DB 4AA7 B196 4A62
PGP Public Key from: keyserver.pgp.com