[PATCH] D57938: [WebAssembly] Update MC for bulk memory
Sam Clegg via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Feb 12 17:23:46 PST 2019
sbc100 added a comment.
In D57938#1395614 <https://reviews.llvm.org/D57938#1395614>, @dschuff wrote:
> Without the context of MC here there's no indication what kind of flags the `.init_flags` directive refers to. Maybe call it `init_segment_flags`?
Better still perhaps we can attach it to the segment/section declaring in the the asm itself? It should be noted that wouter is working changing this right now.. so it probably OK to land this and then consolidate/iterate?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D57938/new/
https://reviews.llvm.org/D57938
More information about the llvm-commits
mailing list