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

Hongbin Zheng etherzhhb at gmail.com
Thu Apr 5 08:00:25 PDT 2012


tobi,

>> +BasicBlockPass *createBBVectorizePass(VectorizeConfig C =
>> VectorizeConfig());
>
>
> What about making the VectorizeConfig 'const' and passed by reference &C?
I doubt we cant write something like:
BasicBlockPass *createBBVectorizePass(const VectorizeConfig &C =
VectorizeConfig())

Otherwise we need to change all user of "createBBVectorizePass" to
pass the VectorizeConfig explicitly, but i don't like spread the
change.

Anyway i am going to have a try.

best regards
ether



More information about the llvm-commits mailing list