[PATCH] D59737: [WebAssembly] Support WebAssemblyFunctionInfo serialization

Heejin Ahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Mar 24 06:52:39 PDT 2019


aheejin marked an inline comment as done.
aheejin added inline comments.


================
Comment at: lib/Target/WebAssembly/WebAssemblyMachineFunctionInfo.h:59-61
+  // Function properties.
+  bool CFGStackified = false;
+
----------------
arsenm wrote:
> I think adding a new field should be a separate patch
I took it out that part in D59747. The primary reason I included that here was without any field to serialize there was no way to test this CL alone. I added an empty `machineFunctionInfo: {}` in the test here but it's printed without this anyway.


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D59737/new/

https://reviews.llvm.org/D59737





More information about the llvm-commits mailing list