arichardson added inline comments.
================ Comment at: llvm/docs/ReleaseNotes.rst:122 been removed. +* n32 was added to the RV64I datalayout string. ---------------- 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"? 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
