[llvm] Verifier: forbid non-i32/i64 lrint, and non-i64 llrint (PR #70839)

Ramkumar Ramachandra via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 31 11:16:21 PDT 2023


artagnon wrote:

Oh. No, the text description doesn't say so explicitly, but we normally lower llvm.lrint and llvm.llrint to libcalls: how is the libcall to lrint supposed to return anything but i32/i64, and that to llrint supposed to return anything but i64? I suppose we could widen/narrow the type in ISelLowering after getting the result of the libcall, but afaik, we do no such thing. Currently, a non-i32/i64 [l]lrint asserts while lowering on many architectures I tested.

I suppose the commit message requires rewording.

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


More information about the llvm-commits mailing list