[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