r170096.

On 12 December 2012 16:39, Eli Friedman <[email protected]> wrote:
> On Wed, Dec 12, 2012 at 2:21 AM, Andrew Turner <[email protected]> wrote:
>> On Sun, 9 Dec 2012 15:06:54 +1300
>> Andrew Turner <[email protected]> wrote:
>>
>>> I have attached a new patch that updates the tests.
>>
>> Were there any issues with this patch or can it be committed?
>
> +      if (getTriple().getOS() != llvm::Triple::FreeBSD) {
> +        SizeType = UnsignedLong;
> +      }
>
> No braces for simple if statements.
>
> Otherwise, LGTM.
>
> -Eli
> _______________________________________________
> cfe-commits mailing list
> [email protected]
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to