[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