[PATCH] D41517: mmintrin.h documentation fixes and updates

Craig Topper via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jan 8 16:30:41 PST 2018


craig.topper added inline comments.


================
Comment at: lib/Headers/mmintrin.h:1402
 ///
-/// This intrinsic corresponds to the <c> VPSHUFLW / PSHUFLW </c> instruction.
+/// This intrinsic corresponds to the <c> PSHUFLW </c> instruction.
 ///
----------------
dyung wrote:
> craig.topper wrote:
> > This is overly specific
> Just to be clear, when you say this is overly specific, you are saying that it should be replaced with the text "This intrinsic is a utility function and does not correspond to a specific instruction."  Is that correct?
Yeah. I just shortened my comment from _mm_set1_pi32.


https://reviews.llvm.org/D41517





More information about the cfe-commits mailing list