[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