simon_tatham added a comment. In D74333#1869788 <https://reviews.llvm.org/D74333#1869788>, @miyuki wrote:
> vrintx is not mentioned anywhere in the patch. Should it say "llvm.rint" > instead? No, I meant what I said: `vrintx` is the MVE machine instruction that `llvm.rint` already instruction-selects to (which is why I didn't have to mention it anywhere in this patch). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D74333/new/ https://reviews.llvm.org/D74333 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
