[PATCH] D113813: [MachO] Reduce size of Symbol and Defined
Vy Nguyen via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Nov 12 17:13:42 PST 2021
oontvoo added a comment.
In D113813#3128715 <https://reviews.llvm.org/D113813#3128715>, @smeenai wrote:
> This will cause merge conflicts for D113167 <https://reviews.llvm.org/D113167>. I'm happy to wait until that lands, if @oontvoo prefers.
>
> I'll move the asserts to Symbols.cpp if I get positive responses to https://reviews.llvm.org/D113809#3128636.
Thanks! Does anyone want to LGTM D113167 <https://reviews.llvm.org/D113167> ? :)
Otherwise, since your patch seems less controversial, feel free to ship it .. I can resolve the conflicts afterwards - didn't seem like a big one (I think)
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D113813/new/
https://reviews.llvm.org/D113813
More information about the llvm-commits
mailing list