On Wed, May 22, 2019 at 1:54 PM Andrew Stubbs <a...@codesourcery.com> wrote: > > This patch fixes a bug observed on amdgcn in which the Fortran frontend > creates function calls using the 32-bit parameters where they ought to > be 64-bit, resulting in UB. > > The issue is caused by the use of "integer_zero_node" where the > definition of the function calls for "size_zero_node". I presume this > works on other architectures because the types are the same size, or > else because parameters are always 64-bit wide. > > OK to commit? > > Andrew
Thanks for the catch. Though for size_t you should use build_zero_cst (size_type_node). size_zero_node is a zero constant of type sizetype, which is not the same as size_type_node (size_t in C). Ok with that change. -- Janne Blomqvist
