[flang-commits] [flang] [flang] In AllocMemOp lowering, convert types for calling malloc on 32-bit (PR #129308)

[Valentin Clement バレンタイン クレメン via flang-commits]

clementval wrote:

> They're the only tests I found that generate a call to the external `malloc` function (which, on a 32-bit target, is now called with an `i32` instead of an `i64` like before).
> 
> 
> 
> Previously, as far as I can tell, the 32-bit code path wasn't being tested at all (because there's minimal-to-no cross-compiling tests as flang doesn't currently properly support it, and all the builders are 64-bit). WebAssembly is unique in being a "new" target which is still 32-bit (32-bit address space, but it has native support for calculations with 64-bit integers).

Ok. Just to make sure. Flang is not supporting 32-bit target and might never fully support them. You might want to add a disclaimer in your commit message about it.





https://github.com/llvm/llvm-project/pull/129308