[PATCH] D20176: [MemCpyOpt] Use MaxIntSize in byte instead of bit

[Jun Bum Lim via llvm-commits]

junbuml added a comment.

> My comments weren't in opposition to this patch, I was just commenting on peculiarities of the test case. Feel free to post a patch to rename the function as Mendi suggests. You might also investigate why the stores aren't being merged.


Thanks Chad. I see your point. From what you pointed out, I found potential improvement in DAG combine which merge :

  store i16 0, i16* %0
  store i16 0, i16* %1

into

  STRWui

but

  store i16 0, i16* %0
  store i16 0, i16* %1
  store i32 0, i32* %2 

into

  STRHHui
  STRHHui
  STRWui


http://reviews.llvm.org/D20176