[llvm] [WebAssembly] Allocate MCSymbolWasm data on MCContext (PR #85866)
Tim Neumann via llvm-commits
llvm-commits at lists.llvm.org
Mon Apr 1 23:26:53 PDT 2024
TimNN wrote:
@MaskRay: Thanks for the review!
> We don't typically include a full stop in the subject line.
Fixed.
> The shorter `allocateString` looks good to me.
Done, renamed to just `allocateString`.
> Consider summarizing the issue in the description.
Done.
---
I don't have commit access, so it would be great if one of you could merge the PR.
https://github.com/llvm/llvm-project/pull/85866
More information about the llvm-commits
mailing list