<div dir="ltr">Looks good to me, though I think LoopVectorize.cpp is getting a bit too big... If we repeat this pattern for other instructions, it'll be flooded with little classes even before it gets to the vectorizer.<div>
<br></div><div style>Nadav, do you think it's time to separate LoopVectorize into Cost / Legal / Vect? Or maybe we should wait a bit more?</div><div style><br></div><div style>Implementation-wise, though, I think it's good to go.</div>
<div style><br></div><div style>cheers,</div><div style>--renato</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On 4 February 2013 20:49, Arnold Schwaighofer <span dir="ltr"><<a href="mailto:aschwaighofer@apple.com" target="_blank">aschwaighofer@apple.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">No you are not wrong. Fixed in the attached patch.<br>
<br>
<br><br>
<br>
<br>
On Feb 4, 2013, at 1:50 PM, Renato Golin <<a href="mailto:renato.golin@linaro.org">renato.golin@linaro.org</a>> wrote:<br>
<br>
> Hi Arnold,<br>
><br>
> I may be wrong, but I think these guys are only ever used internally:<br>
><br>
> +    unsigned getOpcode() { return Opcode; }<br>
> +<br>
> +    unsigned getAlignment() { return Alignment; }<br>
> +<br>
> +    unsigned getPointerAddressSpace() { return AddressSpace; }<br>
> +<br>
> +    Value *getPointerOperand() { return PointerOperand; }<br>
> +<br>
> +    Type *getScalarType() { return ScalarTy; }<br>
> +<br>
> +    Type *getVectorType() { return VectorTy; }<br>
><br>
> So, they don't need the getters, and you can use them directly in the other members of the MemoryOpCost class.<br>
><br>
> cheers,<br>
> --renato<br>
<br></blockquote></div><br></div>