On Fri, Sep 14, 2012 at 12:26 AM, Chandler Carruth <[email protected]> wrote: > LGTM, thanks for the explanation and the comments! > > Also, yay test cases! > > One grammar nit: > > + // targets. It contains N32 ABI binaries. Do not > + // use this folder if produce code for other ABI. > > "Do not use this folder when producing code for the normal ABI."? However > you want to phrase "normal ABI", I'm assuming you mean anything other than > N32 ABI.
Thanks for your review. Committed at r163898. http://llvm.org/viewvc/llvm-project?rev=163898&view=rev -- Simon _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
