[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