[llvm] r254995 - [WebAssembly] Assert MRI.isSSA() in passes that depend on SSA form.
Dan Gohman via llvm-commits
llvm-commits at lists.llvm.org
Mon Dec 7 19:30:43 PST 2015
Author: djg
Date: Mon Dec 7 21:30:42 2015
New Revision: 254995
URL: http://llvm.org/viewvc/llvm-project?rev=254995&view=rev
Log:
[WebAssembly] Assert MRI.isSSA() in passes that depend on SSA form.
Modified:
llvm/trunk/lib/Target/WebAssembly/WebAssemblyRegStackify.cpp
llvm/trunk/lib/Target/WebAssembly/WebAssemblyStoreResults.cpp
Modified: llvm/trunk/lib/Target/WebAssembly/WebAssemblyRegStackify.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/WebAssembly/WebAssemblyRegStackify.cpp?rev=254995&r1=254994&r2=254995&view=diff
==============================================================================
--- llvm/trunk/lib/Target/WebAssembly/WebAssemblyRegStackify.cpp (original)
+++ llvm/trunk/lib/Target/WebAssembly/WebAssemblyRegStackify.cpp Mon Dec 7 21:30:42 2015
@@ -127,6 +127,8 @@ bool WebAssemblyRegStackify::runOnMachin
WebAssemblyFunctionInfo &MFI = *MF.getInfo<WebAssemblyFunctionInfo>();
AliasAnalysis &AA = getAnalysis<AAResultsWrapperPass>().getAAResults();
+ assert(MRI.isSSA() && "RegStackify depends on SSA form");
+
// Walk the instructions from the bottom up. Currently we don't look past
// block boundaries, and the blocks aren't ordered so the block visitation
// order isn't significant, but we may want to change this in the future.
Modified: llvm/trunk/lib/Target/WebAssembly/WebAssemblyStoreResults.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/WebAssembly/WebAssemblyStoreResults.cpp?rev=254995&r1=254994&r2=254995&view=diff
==============================================================================
--- llvm/trunk/lib/Target/WebAssembly/WebAssemblyStoreResults.cpp (original)
+++ llvm/trunk/lib/Target/WebAssembly/WebAssemblyStoreResults.cpp Mon Dec 7 21:30:42 2015
@@ -72,6 +72,8 @@ bool WebAssemblyStoreResults::runOnMachi
const MachineRegisterInfo &MRI = MF.getRegInfo();
MachineDominatorTree &MDT = getAnalysis<MachineDominatorTree>();
+ assert(MRI.isSSA() && "StoreResults depends on SSA form");
+
for (auto &MBB : MF) {
DEBUG(dbgs() << "Basic Block: " << MBB.getName() << '\n');
for (auto &MI : MBB)
More information about the llvm-commits
mailing list