[PATCH] [GCCJIT] support dynamic alloca stub

2024-11-09 Thread Schrodinger ZHU Yifan
This patch adds dynamic alloca stubs support to GCCJIT. DEF_BUILTIN_STUB only define the enum for builtins instead of providing the type. Therefore, builtins with stub will lead to ICE before this patch. This applies to `alloca_with_align`, `stack_save` and `stack_restore`. This patch add special

Re: [PATCH] [GCCJIT] support dynamic alloca stub

2024-11-09 Thread Schrodinger ZHU Yifan
Thank you for the quick review. Indeed, I reverted the changes to tree-ssa-ccp.cc and applied your changes. All tests still pass. Schrodinger ZHU Yifan, Ph.D. Student Computer Science Department, University of Rochester Personal Email: i...@zhuyi.fan Work Email: yifan...@rochester.edu Website

Re: [PATCH v2] gccjit: support dynamic alloca stub

2024-11-10 Thread Schrodinger ZHU Yifan
sorry, just noticed that the check_value was wrongly written on the last line of this patch. Other parts should be good. Should I correct it now in a new revision? Or is it okay to wait for further reviews for now? On Sun, Nov 10, 2024 at 13:17, Schrodinger ZHU Yifan <i...@zhuyi.fan>

[PATCH v3] [GCCJIT] support dynamic alloca stub

2024-11-10 Thread Schrodinger ZHU Yifan
This patch adds dynamic alloca stubs support to GCCJIT. DEF_BUILTIN_STUB only defines the enum for builtins instead of providing the type. Therefore, builtins with stub will lead to ICE before this patch. This applies to `alloca_with_align`, `stack_save` and `stack_restore`. This patch adds speci

[PATCH v2] gccjit: support dynamic alloca stub

2024-11-10 Thread Schrodinger ZHU Yifan
This patch adds dynamic alloca stubs support to GCCJIT. DEF_BUILTIN_STUB only defines the enum for builtins instead of providing the type. Therefore, builtins with stub will lead to ICE before this patch. This applies to `alloca_with_align`, `stack_save` and `stack_restore`. This patch adds speci