[PATCH] D82097: WebAssembly: Don't store MachineFunction in MachineFunctionInfo
Matt Arsenault via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Jun 18 07:35:24 PDT 2020
arsenm created this revision.
arsenm added reviewers: sunfish, tlively, aheejin, dschuff.
Herald added subscribers: hiraditya, jgravelle-google, sbc100, wdng.
Herald added a project: LLVM.
Soon it will be disallowed to depend on MachineFunction state in the
constructor. This was only being used to get the MachineRegisterInfo
for an assert, which I'm not sure is necessarily worth it. I would
think any missing defs would be caught by the verifier later instead.
https://reviews.llvm.org/D82097
Files:
llvm/lib/Target/WebAssembly/WebAssemblyExplicitLocals.cpp
llvm/lib/Target/WebAssembly/WebAssemblyLowerBrUnless.cpp
llvm/lib/Target/WebAssembly/WebAssemblyMachineFunctionInfo.cpp
llvm/lib/Target/WebAssembly/WebAssemblyMachineFunctionInfo.h
llvm/lib/Target/WebAssembly/WebAssemblyPeephole.cpp
llvm/lib/Target/WebAssembly/WebAssemblyRegNumbering.cpp
llvm/lib/Target/WebAssembly/WebAssemblyRegStackify.cpp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D82097.271709.patch
Type: text/x-patch
Size: 7048 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200618/52faf0d6/attachment.bin>
More information about the llvm-commits
mailing list