[llvm-commits] [Review Request][Patch]Introduce the VectorizeConfig class.

[Hal Finkel]

On Thu, 5 Apr 2012 23:03:49 +0800
Hongbin Zheng <etherzhhb at gmail.com> wrote:

> On Thu, Apr 5, 2012 at 11:00 PM, Hongbin Zheng <etherzhhb at gmail.com>
> wrote:
> > tobi,
> >
> >>> +BasicBlockPass *createBBVectorizePass(VectorizeConfig C =
> >>> VectorizeConfig());
> >>
> >>
> >> What about making the VectorizeConfig 'const' and passed by
> >> reference &C?
> It works :P and i will apply this to the patch

Good. With this change, the patch looks good to me. Thanks for working
on this.

 -Hal

> >
> > best regards
> > ether



-- 
Hal Finkel
Postdoctoral Appointee
Leadership Computing Facility
Argonne National Laboratory