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

Reply via email to