[PATCH] D96471: [SystemZ] Fix vecintrin.h to not emit alignment hints in vec_xl/vec_xst.

Ulrich Weigand via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 11 10:53:36 PST 2021


uweigand added a comment.

It's not a good idea to use the library routine "memcpy" here, because the name in theory be redefined by a user before including the header, and also there may not be a prototype in scope causing warnings when building with -Wall.

But both problems can be fixed by simply using the intrinsic "__builtin_memcpy" instead.

Patch LGTM with that change.


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

https://reviews.llvm.org/D96471



More information about the llvm-commits mailing list