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

Saleem Abdulrasool via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 19 10:06:07 PST 2018


compnerd added a comment.

What is the motivation for this change?  If you are trying to enable this for MinGW, that is fine, but I'm not sure if we should try to catch the VLA issues in the frontend.  I believe that on x86_64, we also disallow the VLAs as Microsoft does not permit them there, but @rnk can correct me if I'm wrong on that.


https://reviews.llvm.org/D42292





More information about the llvm-commits mailing list