[llvm-commits] [cfe-commits] The AArch64 LLVM (& Clang) target

[David Blaikie]

On Mon, Jan 7, 2013 at 6:04 AM, Tim Northover <Tim.Northover at arm.com> wrote:
> Hi Dmitri,
>
> Thanks for your comments. I'll fix the issues before sending an updated patch
> (I won't send immediately, the patches are quite large and I don't want to
> Spam the list).

Perhaps using the Phabricator code review (
http://llvm-reviews.chandlerc.com/ ) would avoid some of the pain of
sending large patch attachments to the list repeatedly. (or maybe even
a github clone, etc - not sure about that though)

>
> On Monday 07 Jan 2013 13:56:23 you wrote:
>> On Mon, Jan 7, 2013 at 2:57 PM, Tim Northover <Tim.Northover at arm.com> wrote:
>> +/// UnknownPadding - Return the worst case padding that could result from
>
>> When adding new code, please follow the new guidelines about doxygen
>> documentation: don't duplicate function and class names in the
>> comment.  Thank you.
>
> Of course. I thought I'd caught all of these at the end of last year. I may
> well have forgotten Clang, I'll check again.
>
> Regards.
>
> Tim.
>
> -- IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium.  Thank you.
>
>
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits