On Wed, 10 Apr 2024 10:13:37 GMT, Joachim Kern <jk...@openjdk.org> wrote:

>> Can `-Dalloca=__builtin_alloca` replace `#include <alloca.h>`?
>
> Yes I believe. I will remove the `#pragma alloca` everywhere, I will remove 
> the `#include <alloca.h>` everywhere and I will add  
> `-Dalloca=__builtin_alloca` to the compile commands. If it works I will 
> update the PR.

In my humble opinion the inclusion of alloca.h was slightly cleaner, but I 
guess it doesn't matter. Out of curiosity, why do you guys prefer not including 
it?

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/18536#discussion_r1559450230

Reply via email to