[PATCH] D13771: [AArch64]Add support for converting halfword loads into a 32-bit word load

[Jun Bum Lim via llvm-commits]

junbuml added a comment.

- I agree that implementing this in the separate loop will open up more pairing.
- Okay, I will append MMO.
- Yes, for low 16, AND should be preferred.

Thanks Chad for the review.


================
Comment at: lib/Target/AArch64/AArch64LoadStoreOptimizer.cpp:168
@@ +167,3 @@
+  case AArch64::LDRHHui:
+    return true;
+  case AArch64::LDURHHi:
----------------
mcrosier wrote:
> Just delete and fall through.
Thanks. I will fix it.


http://reviews.llvm.org/D13771