[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:53:02 PDT 2023


artagnon wrote:

Moreover, as evident from the patch, there is no test coverage for non-i32/i64 llvm.lrint and non-i64 llvm.llrint in test/CodeGen for any target. If future contributors want to support these things, they can remove the Verifier check, and add test coverage along with their patch. As it currently stands, I'm convinced that we don't support these configurations in practice, even if the LangRef isn't explicit about this.

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


More information about the llvm-commits mailing list