[PATCH] D124067: [x86] Support 3 builtin functions for 32-bits targets

[Xiang Zhang via Phabricator via cfe-commits]

xiangzhangllvm marked an inline comment as done.
xiangzhangllvm added inline comments.


================
Comment at: clang/lib/Headers/emmintrin.h:3476
+/// This intrinsic corresponds to the <c> VMOVQ / MOVQ </c> instruction
+/// in 64 bits.
 ///
----------------
RKSimon wrote:
> xiangzhangllvm wrote:
> > craig.topper wrote:
> > > craig.topper wrote:
> > > > 64 bits -> 64-bit mode
> > > 64 bits -> 64-bit
> > Thank you!
> 64-bit (missing hypen)
Let me directly add it when push.
And I see a test failed duo to clang-format. I'll commit a patch only do clang format for this 2 tests. (NFC)
Thank you !


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D124067/new/

https://reviews.llvm.org/D124067