[PATCH] [SelectionDAG] Move RegsForValue into SelectionDAGBuilder.h. NFC.

Sanjoy Das sanjoy at playingwithpointers.com
Mon May 4 16:27:55 PDT 2015


On Mon, May 4, 2015 at 11:53 AM, Andrew Trick <atrick at apple.com> wrote:
> LGTM with one comment...
>
> I'm guessing there isn't a good reason for the constructor to be defined inline:
> RegsForValue(LLVMContext &Context, const TargetLowering &tli,

Will move the constructors to SelectionDAGBuilder.cpp before check-in.

Thanks for the quick reviews!

>
>   unsigned Reg, Type *Ty) {}
>
>
> http://reviews.llvm.org/D9478
>
> EMAIL PREFERENCES
>   http://reviews.llvm.org/settings/panel/emailpreferences/
>
>



More information about the llvm-commits mailing list