efriedma added inline comments.
================
Comment at: clang/lib/CodeGen/CGBuiltin.cpp:10331
+ auto Alignment = CGM.getNaturalPointeeTypeAlignment(
+ E->getArg(0)->IgnoreParenCasts()->getType());
Ops[0] = Builder.CreateBitCast(Ops[0], llvm::PointerType::getUnqual(VTy));
----------------
IgnoreParenCasts() here seems really dubious. You could easily end up with a
type that's not even a pointer.
Can we just set the alignment to 1 and leave it at that? It's not like it
really matters for NEON.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D79721/new/
https://reviews.llvm.org/D79721
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
