[flang-commits] [flang] [Flang][Runtime] Fix implicit conversion warning when targeting 32bit… (PR #99465)

via flang-commits flang-commits at lists.llvm.org
Fri Jul 19 09:33:18 PDT 2024


https://github.com/jeanPerier commented:

> I assume that for a 32 bit target, we should never meet an overflow in subscript computation.

It seems a correct assumption to me, SubscriptValue should just be 64bit on this target if one intended to support large susbcripts on 32bit target. Although as @klausler mentioned 32bit targets have not been tested by us.

> A few words on the motivation : there are a few projects around there that compiles Lapack to wasm through a modified flang, e.g. https://github.com/r-wasm/flang-wasm . I'm trying to upstream the parts that could be relevant / beneficial to flang as a whole, without being too intrusive.

Thanks for that. 

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


More information about the flang-commits mailing list