aaron.ballman accepted this revision.
aaron.ballman added a comment.

The header bits LGTM aside from a nit I found with potential identifier 
conflicts. I don't feel super qualified to review the behavioral aspects of the 
change, but if others are happy with that, I'm happy with the header changes.



================
Comment at: clang/lib/Headers/mm_malloc.h:31-32
 static __inline__ void *__attribute__((__always_inline__, __nodebug__,
-                                       __malloc__))
-_mm_malloc(size_t __size, size_t __align)
-{
+                                       __malloc__, alloc_size(1),
+                                       alloc_align(2)))
+_mm_malloc(size_t __size, size_t __align) {
----------------
Protect against user macros.


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

https://reviews.llvm.org/D117091

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to