[PATCH] D56091: [WebAssembly] Don't add IMPLICIT_DEFs in PrepareForLiveIntervals

Heejin Ahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 26 14:23:06 PST 2018


aheejin created this revision.
aheejin added reviewers: dschuff, sunfish.
Herald added subscribers: llvm-commits, jgravelle-google, sbc100, MatzeB.

Not sure about the situation when this pass was first added, but uses
without defs are generated from IMPLICIT_DEF instructions from
instruction selection phase correctly now, and all IMPLICIT_DEFs
generated by this pass result in dead live ranges that start and end in
the same instruction. So the only thing we need to do with this pass is
to make sure we track the liveness again.

All test changes are local number changes, which are not meaningful.


Repository:
  rL LLVM

https://reviews.llvm.org/D56091

Files:
  lib/Target/WebAssembly/WebAssemblyOptimizeLiveIntervals.cpp
  lib/Target/WebAssembly/WebAssemblyPrepareForLiveIntervals.cpp
  test/CodeGen/WebAssembly/reg-stackify.ll
  test/CodeGen/WebAssembly/umulo-128-legalisation-lowering.ll
  test/CodeGen/WebAssembly/umulo-i64.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D56091.179524.patch
Type: text/x-patch
Size: 8734 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20181226/59541370/attachment.bin>


More information about the llvm-commits mailing list