[llvm-commits] Minor change to LoopUnroll's ctor

Junjie Gu jgu222 at gmail.com
Mon Apr 11 13:44:14 PDT 2011


Hello,

Is this change okay ?

Thanks
Junjie

On Mon, Apr 4, 2011 at 1:47 PM, Junjie Gu <jgu222 at gmail.com> wrote:
> I'd like to change the loop unroll so that the
> threshold/count/allowpartial can be passed as parameters to
> LoopUnroll's ctor. The reason for doing so is that static options
> (-unroll-threshold, -unroll-count, -unroll-allow-partial) are not good
> for an environment (OpenCL) in which multiple llvm context may be
> present at the same time. And each llvm context may have its own
> optimization options, and thus options cannot be shared.
>
> With this patch, a user can create a loop unroll by the following:
>
>          createLoopUnrollPass(Threshold, Count, AllowPartial);
>
> And the previous API
>          createLoopUnrollPass()
> will work as before.
>
> Please review it. I do have commit access so I can check in by myself
> after reviewing.
>
> Thanks
> Junjie
>




More information about the llvm-commits mailing list