[PATCH] D39403: [LV][X86] update the cost of interleaving mem. access of floats

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 1 13:32:05 PDT 2017


RKSimon added a comment.

In https://reviews.llvm.org/D39403#912689, @magabari wrote:

> In https://reviews.llvm.org/D39403#912670, @RKSimon wrote:
>
> > Didn't @m_zuckerman put the previous tests into llvm/test/Analysis/CostModel/interleaved-*.ll ? Maybe match that?
>
>
> Do you mean renaming the test to interleaved-load-float.ll?
>  because as i see those tests is for i8 and i32 while i am checking new datatype.


Yes please, move it to llvm/test/Analysis/CostModel/ and rename it interleaved-load-float.ll


https://reviews.llvm.org/D39403





More information about the llvm-commits mailing list