[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