This revision was automatically updated to reflect the committed changes.
Closed by commit rG974e2e690b40: [RISCV] Adjust RV64I data layout by using
n32:64 in layout string (authored by craig.topper).
Changed prior to commit:
https://reviews.llvm.org/D116735?vs=471385&id=471564#toc
Repository:
frasercrmck accepted this revision.
frasercrmck added a comment.
LGTM other than a nit, but I concur that a comment in AutoUpgrade would be nice.
Comment at: llvm/docs/ReleaseNotes.rst:123
+* i32 is now a native type in the datalayout string. This enables
+ LoopStrengthReduce
craig.topper updated this revision to Diff 471385.
craig.topper added a comment.
Refine ReleaseNotes
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D116735/new/
https://reviews.llvm.org/D116735
Files:
clang/lib/Basic/Targets/RISCV.h
llvm/docs/Re
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
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 descrip
