[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