[llvm-commits] [PATCH] LTO code generator options

Viktor Kutuzov vkutuzov at accesssoftek.com
Tue Nov 17 12:26:02 PST 2009


Thanks for reviewing it, Rafael.

Now while we are both on the same waive, let's go with the rest of the patches.
I have tried to combine them on the same manner as the last one.

The attached patch has similar OS and Vendor independent re-factoring for Subtarget features (attributes).

I moved Applce-specific code from tools/lto/LTOModule.cpp to lib/Target/SubtargetFeature.cpp with related header files update.

Cheers,
Viktor.

________________________________________
From: Rafael Espindola [espindola at google.com]
Sent: Tuesday, November 17, 2009 8:09 AM
To: Viktor Kutuzov
Cc: Commit Messages and Patches for LLVM
Subject: Re: [llvm-commits] [PATCH] LTO code generator options

2009/11/16 Viktor Kutuzov <vkutuzov at accesssoftek.com>:
> Please find the attached patch.
> Does this look like what you want to see?
>
> I have moved the check for Darwin OS insode the getArchNameForAssembler which makes it OS and Vendor independent. This is what we wanted from the beginning, aren't we?
> And added missed ppc64. Thanks for catching this.

The patch looks good to me!

Thanks a lot.

> Thanks,
> Viktor.

Cheers,
--
Rafael Ávila de Espíndola
-------------- next part --------------
A non-text attachment was scrubbed...
Name: llvm-lto-codegen-subtargetfeature-default_feature.diff
Type: text/x-patch
Size: 4471 bytes
Desc: llvm-lto-codegen-subtargetfeature-default_feature.diff
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20091117/9f03d6be/attachment.bin>


More information about the llvm-commits mailing list