LGTM.

-Eli


On Fri, Sep 20, 2013 at 9:45 AM, Petar Jovanovic <[email protected]
> wrote:

>  Any additional comments? Ok to commit?
>
>  Regards,
> Petar
>  ------------------------------
> *From:* Petar Jovanovic
> *Sent:* Friday, September 20, 2013 12:47 AM
> *To:* Eli Friedman
>
> *Cc:* [email protected]; [email protected]; Eli Bendersky
> *Subject:* RE: [PATCH] Recognize llvm::Triple::NaCl for MIPSEL
>
>   The patch has been updated with the test case. Take a look.
>
>  Regards,
> Petar
>  ------------------------------
> *From:* Eli Friedman [[email protected]]
> *Sent:* Thursday, September 19, 2013 12:38 AM
> *To:* Petar Jovanovic
> *Cc:* [email protected]; [email protected]; Eli Bendersky
> *Subject:* Re: [PATCH] Recognize llvm::Triple::NaCl for MIPSEL
>
>   On Wed, Sep 18, 2013 at 11:28 AM, Petar Jovanovic <
> [email protected]> wrote:
>
>> A patch to AllocateTarget to recognize llvm::Triple::NaCl for MIPSEL and
>> return NaClTargetInfo. Same as what other architectures in NaCl already
>> do.
>>
>
>  Missing testcase?
>
>  -Eli
>
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to