[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