Page MenuHomePhabricator

Please use GitHub pull requests for new patches. Avoid migrating existing patches. Phabricator shutdown timeline

Differential D68666

LiveIntervals: Split live intervals on multiple dead defs
ClosedPublic
Actions

Authored by kparzysz on Oct 8 2019, 2:23 PM.

Details

Reviewers
qcolombet
arsenm
Commits
rG43144ffa91a2: LiveIntervals: Split live intervals on multiple dead defs
Summary

This is a follow-up to D67448.

Split live intervals with multiple dead defs during the initial execution of the live interval analysis, but do it outside of the function createAndComputeVirtRegInterval.

Diff Detail

Event Timeline

kparzysz created this revision.Oct 8 2019, 2:23 PM
Herald added a project: Restricted Project. · View Herald TranscriptOct 8 2019, 2:23 PM
Herald added subscribers: aheejin, jgravelle-google, sbc100 and 5 others. · View Herald Transcript
arsenm accepted this revision.Oct 16 2019, 12:37 PM

LGTM

This revision is now accepted and ready to land.Oct 16 2019, 12:37 PM
Closed by commit rG43144ffa91a2: LiveIntervals: Split live intervals on multiple dead defs (authored by kparzysz). · Explain WhyOct 30 2019, 6:51 AM
This revision was automatically updated to reflect the committed changes.
Herald added a subscriber: hiraditya. · View Herald TranscriptOct 30 2019, 6:51 AM
foad mentioned this in D151227: RFC: LiveIntervalCalc: Always split separate components.May 23 2023, 9:24 AM

Revision Contents
Changeset List

PathSize
llvm/
include/
llvm/
CodeGen/
2 lines
lib/
CodeGen/
17 lines
test/
DebugInfo/
WebAssembly/
8 lines
X86/
5 lines

Diff 227085

View Options

llvm/include/llvm/CodeGen/LiveIntervals.h

Loading...
View Options

llvm/lib/CodeGen/LiveIntervals.cpp

Loading...
View Options

llvm/test/DebugInfo/WebAssembly/dbg-value-move-reg-stackify.mir

Loading...
View Options

llvm/test/DebugInfo/X86/live-debug-vars-discard-invalid.mir

Loading...