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

Simon Pilgrim via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Apr 21 03:56:44 PDT 2022


RKSimon accepted this revision.
RKSimon added a comment.
This revision is now accepted and ready to land.

LGTM with one minor typo



================
Comment at: clang/lib/Headers/emmintrin.h:3476
+/// This intrinsic corresponds to the <c> VMOVQ / MOVQ </c> instruction
+/// in 64 bits.
 ///
----------------
xiangzhangllvm wrote:
> craig.topper wrote:
> > craig.topper wrote:
> > > 64 bits -> 64-bit mode
> > 64 bits -> 64-bit
> Thank you!
64-bit (missing hypen)


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

https://reviews.llvm.org/D124067



More information about the cfe-commits mailing list