Argh, sorry for the confusion, I'll never get used to llvm's AllCaps style
for variables :)


On Thu, Jan 3, 2013 at 4:51 PM, Vane, Edwin <[email protected]> wrote:

> TransformManager is of type 'Transforms' and Transforms.h and
> Transforms.cpp are both present in the diff. Although, I like the name
> TransformManager better so maybe I should rename the class just to
> differentiate it from 'Transform' by more than just an 's'?
>
> Everything under LoopConvert is a copy except LoopConvert.h and
> LoopConvert.cpp. LoopConvert.cpp comes mostly from
> loop-convert/LoopConvert.cpp just with simplifications due to the new
> location in a class. Actually if you dir-diffed loop-convert and
> cpp11-migrate/LoopConvert you would see exactly the changes I made. There
> were a few stylistic things I improved.
>
> -----Original Message-----
> From: Manuel Klimek [mailto:[email protected]]
> Sent: Thursday, January 03, 2013 10:29 AM
> To: [email protected]; [email protected]; Vane, Edwin
> Cc: [email protected]
> Subject: Re: [PATCH] Port loop-convert into cpp11-migrate
>
>
>   Is the TransformManager implementation missing from the diff?
>
>   Also, can you point out which files are pure copies, so I know I don't
> need to re-review them? Thanks!
>
> http://llvm-reviews.chandlerc.com/D251
>
> _______________________________________________
> 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