[PATCH] D125515: [WebAssembly] Fix register use-def in FixIrreducibleControlFlow

Heejin Ahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 12 20:17:53 PDT 2022


aheejin added inline comments.


================
Comment at: llvm/lib/Target/WebAssembly/WebAssemblyReplacePhysRegs.cpp:75-77
-  // We don't preserve SSA or liveness.
-  MRI.leaveSSA();
-  MRI.invalidateLiveness();
----------------
I'm not sure why these were here. This pass replaces physical registers with virtual ones, and this is one-to-one relationship so I think this pass doesn't change anything in liveness or SSA.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D125515/new/

https://reviews.llvm.org/D125515



More information about the llvm-commits mailing list