[PATCH] D57938: [WebAssembly] Update MC for bulk memory

Thomas Lively via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 15 16:02:38 PST 2019


tlively added inline comments.


================
Comment at: llvm/lib/Target/WebAssembly/MCTargetDesc/WebAssemblyTargetStreamer.h:20
 #include "llvm/MC/MCStreamer.h"
+#include "llvm/MC/MCSectionWasm.h"
 #include "llvm/Support/MachineValueType.h"
----------------
dschuff wrote:
> You don't need to include this here, you can just forward declare `class MCSectionWasm` down on line26 with the others.
This was no longer necessary at all, so I removed it.


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