[PATCH] D52586: [X86] Add the movbe instruction intrinsics from icc.

[Sanjay Patel via Phabricator via cfe-commits]

spatel added a comment.

The struct hack isn't obvious to me. Without that, we would produce a load with default alignment based on the size of the load (i132 -> align 4, etc)? But we want to force align 1 regardless of the load size, so the __packed__ attribute on the struct gets us that IIUC. What does __may_alias__ do?

Explain this in a code comment in the header to make this less tricky?



================
Comment at: lib/Headers/immintrin.h:359
+#endif
+#endif /* __MOVEBE */
+
----------------
MOVEBE -> MOVBE


https://reviews.llvm.org/D52586