[PATCH] D42292: [ARM] Call __chkstk for dynamic stack allocation in all windows environments

[Martin Storsjö via Phabricator via llvm-commits]

mstorsjo added a comment.

In https://reviews.llvm.org/D42292#985281, @compnerd wrote:

> I'm not sure that I follow how this is only for the `alloca` function.  It is for an intrinsic, which just generally enables it.  Does it match a cl intrinsic?


Yes - this is for an intrinsic, which enables it for both the `alloca` function and for VLAs - on the LLVM level they're indistinguishable. MSVC supports the alloca function.

If you want to disallow VLAs, that has to be done in clang, not here.


https://reviews.llvm.org/D42292