glider added a comment.

I second this proposal.
https://reviews.llvm.org/D60548 suggests to handle this case using a pragma, 
but the required change seems to be more intrusive.



================
Comment at: clang/lib/CodeGen/CGBuiltin.cpp:3436
 
   case Builtin::BI__builtin_alloca_with_align: {
     Value *Size = EmitScalarExpr(E->getArg(0));
----------------
For the sake of completeness, shall we also implement an uninitialized version 
of __builtin_alloca_with_align()?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D115440

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

Reply via email to