[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