[cfe-commits] [PATCH]Add PNaCl TargetInfo to Clang

Ivan Krasin krasin at chromium.org
Wed Aug 24 13:07:36 PDT 2011


On Wed, Aug 24, 2011 at 12:51 PM, Eric Christopher <echristo at apple.com> wrote:
>
> On Aug 24, 2011, at 12:36 PM, Ivan Krasin wrote:
>
>> Hi Eric,
>>
>> Please, find the updated patch attached or
>> http://codereview.chromium.org/7717004/
>>
>> 1. I've updated RUN: lines as suggested. Now, it does not create temp files
>
> Yeah, we try to avoid grep if at all possible :)
Done.

>
>> 2. I've fixed __builtin_va_list definition and added a test for that
>> (unrelated catch)
>
> Looks good. Thanks!
>
> -eric
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: pnacl_target_no_grep.patch
Type: text/x-patch
Size: 5375 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20110824/9dfe4d11/attachment.bin>


More information about the cfe-commits mailing list