[PATCH] Fix clash of gcc toolchains in driver regression tests

Rafael Espíndola rafael.espindola at gmail.com
Thu Sep 11 06:49:08 PDT 2014


On 10 September 2014 19:02, Samuel F Antao <sfantao at us.ibm.com> wrote:
> Hi all,
>
> Some regression tests for the driver rely on the minimal/dummy system root
> that comes in the Inputs folder included in the clang driver tests’s folder.
> The path to the right toolchain is specified using the —sysroot switch.
> However, if clang was configured with a custom gcc toolchain (either by
> using GCC_INSTALL_PREFIX in cmake or the equivalent configure command), the
> path to the custom gcc toolchain path takes precedence to the one specified
> by sysroot. This causes several regression tests to fail as they will be
> using an unexpected path.
> This patch fixes this issue by adding --gcc-toolchain=“” to all tests that
> rely on that. The empty string causes the driver to pick the path from
> sysroot instead. There aren’t any changes in the driver’s code, only the
> tests are modified. The driver's regression tests for hexagon also have
> similar issues but these require modifications in the driver’s code. I’ll
> send these fixes in a separate patch.

I think this is OK, but Chandler should probably have the final say.

Cheers,
Rafael




More information about the cfe-commits mailing list