[PATCH] D42441: IRGen: Emit an inline implementation of __builtin_wmemcmp on MSVCRT platforms.

[David Majnemer via Phabricator via llvm-commits]

majnemer added inline comments.


================
Comment at: clang/lib/CodeGen/CGBuiltin.cpp:1771-1774
+    Value *DstCh =
+        Builder.CreateAlignedLoad(WCharTy, DstPhi, CharUnits::fromQuantity(2));
+    Value *SrcCh =
+        Builder.CreateAlignedLoad(WCharTy, SrcPhi, CharUnits::fromQuantity(2));
----------------
Should we be more explicit about where 2 comes from? Maybe `getTypeAlignInChars(WCharTy)`?


https://reviews.llvm.org/D42441