[PATCH] D154490: [libunwind] Add missing alloca.h include

Steven van der Schoot via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 5 06:44:46 PDT 2023


StevenvdSchoot updated this revision to Diff 537344.
StevenvdSchoot edited the summary of this revision.
StevenvdSchoot added a comment.
Herald added subscribers: abidh, krytarowski.

In D154490#4473310 <https://reviews.llvm.org/D154490#4473310>, @mstorsjo wrote:

> `alloca.h` isn’t a standard header, and not all those platforms included in the ifdef do have a header with that name (even if they have the `alloca` function/macro provided somehow by some other header).
>
> See earlier discussion in e.g. https://reviews.llvm.org/D149635.

Ahh, sorry I missed that PR.
I see there has been no activity on that issue since May 10. Based on the comments there, I created a new version.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D154490

Files:
  libunwind/src/config.h

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D154490.537344.patch
Type: text/x-patch
Size: 7855 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20230705/f65d84f1/attachment.bin>


More information about the llvm-commits mailing list