On Jan 15, 2021, David Malcolm <dmalc...@redhat.com> wrote: > I think an issue here was that I assumed check_effective_target_alloca > checks that "alloca" is supported, whereas I now see that I was wrong; > it actually checks for "__builtin_alloca".
Yeah, it's a little misleading. And it only tests __builtin_alloca on nvptx, it's assumed present on all other targets. Almost a no-op. > I have no objections to the patch. Thanks, I put it in. -- Alexandre Oliva, happy hacker https://FSFLA.org/blogs/lxo/ Free Software Activist GNU Toolchain Engineer Vim, Vi, Voltei pro Emacs -- GNUlius Caesar
