[llvm] [WebAssembly] Allocate MCSymbolWasm data on MCContext. (PR #85866)

Tim Neumann via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 1 09:26:29 PDT 2024


TimNN wrote:

> Regarding who reviews: MCContext is one of those parts of the code where there are many stakeholders, but no real "owner" so it's hard to get things reviewed sometimes. Maybe having me approve this will get someone's attention? :)

@dschuff: I'm gonna interpret that as "I'd prefer someone else to take a second look at the MCContext parts". (If I misinterpreted that and you're fine with merging as-is, could you trigger the merge? I don't have commit permissions).

Thank you for your review! 

---

@MaskRay: Since you are part of `pr-subscribers-mc`, would you be an appropriate reviewer of the `MCContext` portion of this PR (or can you recommend someone else)?

https://github.com/llvm/llvm-project/pull/85866


More information about the llvm-commits mailing list