[libcxx-commits] [PATCH] D90615: [libunwind] Add missing #include <alloca.h>

Alexander Richardson via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Thu Mar 25 05:37:22 PDT 2021


arichardson added inline comments.


================
Comment at: libunwind/src/config.h:128
+#include <alloca.h>
 #define _LIBUNWIND_REMEMBER_ALLOC(_size) alloca(_size)
 #define _LIBUNWIND_REMEMBER_FREE(_ptr)                                         \
----------------
Couldn't this simply be changed to `__builtin_alloca()` to avoid the header dependency?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D90615



More information about the libcxx-commits mailing list