jrtc27 added inline comments.
================ Comment at: llvm/docs/ReleaseNotes.rst:122 been removed. +* n32 was added to the RV64I datalayout string. ---------------- arichardson wrote: > Without additional context I don't think this makes much sense to most > readers. Before looking at this patch description I would not have been and > to say what n is used for. > > Maybe something like "i32 has been marked as a legal integer type for RV64, > improving code generation for some benchmarks"? "native integer type" not "legal integer type"; it still gets legalised during ISel Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D116735/new/ https://reviews.llvm.org/D116735 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
