feat(riscv): basic datalayout support - #1231
Conversation
4c2623d to
3b5d144
Compare
naveen-seth
left a comment
There was a problem hiding this comment.
Otherwise, LGTM! I’m still new to the project/this area, so I’ll leave the approval to someone else.
fine to approve (or not) as you choose! we typically wait for all reviewers to approve, or at least most, depending on the complexity of the PR |
math-fehr
left a comment
There was a problem hiding this comment.
I think that's good to me, I don't know how it's done in MLIR/LLVM right now, but you know better!
Co-authored-by: Mathieu Fehr <mathieu.fehr@gmail.com>
you can find MLIR's version here: https://mlir.llvm.org/docs/DataLayout/ the goal in both cases is to provide an interface that answers questions about how objects want to be represented in RAM. there isn't any question that we need to be able to answer these questions, but of course we might disagree on how to best answer them. the answer here is "these are hard-coded, but then exposed through a generic API." this is far less ambitious than what MLIR provides, which includes things like parsing and printing LLVM-style datalayout strings. anyway I'm happy to talk about this, discuss alternatives, etc. but for now, this solves practical problems that we've already encountered -- and solved in ad hoc ways. |
luisacicolini
left a comment
There was a problem hiding this comment.
lgtm, thank you :)
this only does the RISC-V side, but we also want datalayout in the LLVM interpreter since it interacts with memory objects as well