On Sun, Mar 17, 2013 at 10:37 PM, Rafael EspĂ­ndola
<[email protected]> wrote:
> On 17 March 2013 10:52, Hans Wennborg <[email protected]> wrote:
>> 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?

Attaching new patch with FileCheckized test.

OK to commit?

Thanks,
Hans

Attachment: gcc_construct_job_ppc64_fix2.patch
Description: Binary data

_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to