[cfe-commits] r148950 - in /cfe/trunk: include/clang/Driver/Driver.h include/clang/Driver/HostInfo.h lib/Driver/CMakeLists.txt lib/Driver/Driver.cpp lib/Driver/HostInfo.cpp lib/Driver/Tools.cpp

[Sebastian Pop]

On Wed, Jan 25, 2012 at 3:09 PM, Chandler Carruth <chandlerc at gmail.com> wrote:
> However, I'd point out that not a single test or build bot caught this. =/
> At the very least we really need tests for the hexagon toolchain. You can
> lee test/Driver/linux-ld.c and test/Driver/linux-header-search.cpp for
> examples of how you can test a toolchain very thoroughly in the regression

Thanks for the pointers.

> test suite. Without these types of tests, the ongoing refactoring work in
> the driver is very likely to break your config again. =/ A build bot for the
> hexagon stuff would also be amazing, but I understand the logistical
> challenges there.

We actually have a ticket open to add a build bot reporting public results,
although I don't know how quickly we will turn that on.  Until then, I will
report if something breaks our internal tests.

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