[Patch] Fix redundant comparison in gcc::Common::ConstructJob
Rafael EspĂndola
rafael.espindola at gmail.com
Sun Mar 17 15:37:12 PDT 2013
On 17 March 2013 10:52, Hans Wennborg <hans at chromium.org> wrote:
> Hi all,
>
> I found the suspect expression "Arch == llvm::Triple::x86_64 || Arch
> == llvm::Triple::x86_64" in gcc::Common::ConstructJob. It seems the
> right-hand side used to check for powerpc64, but got lost in a
> refactoring.
>
> The attached patch fixes this and tries to update the test too.
The patch itself looks good, but if it is not too much to ask, could
you also change test/Driver/unknown-gcc-arch.c to use FileCheck?
> Please take a look!
>
> Thanks,
> Hans
Thanks,
Rafael
More information about the cfe-commits
mailing list