diff options
| author | Jakob Stoklund Olesen <stoklund@2pi.dk> | 2012-10-16 22:51:58 +0000 |
|---|---|---|
| committer | Jakob Stoklund Olesen <stoklund@2pi.dk> | 2012-10-16 22:51:58 +0000 |
| commit | 320db3f8052c9f506d9ea043ba5da534df40aa08 (patch) | |
| tree | 8940e7352ef9f815644b8a4e4021f0d98be78ee0 /tools | |
| parent | cdcdfd2cab67366b1debbe36bf46c29f7fecda67 (diff) | |
| download | llvm-320db3f8052c9f506d9ea043ba5da534df40aa08.tar.gz llvm-320db3f8052c9f506d9ea043ba5da534df40aa08.tar.bz2 llvm-320db3f8052c9f506d9ea043ba5da534df40aa08.tar.xz | |
Avoid rematerializing a redef immediately after the old def.
PR14098 contains an example where we would rematerialize a MOV8ri
immediately after the original instruction:
%vreg7:sub_8bit<def> = MOV8ri 9; GR32_ABCD:%vreg7
%vreg22:sub_8bit<def> = MOV8ri 9; GR32_ABCD:%vreg7
Besides being pointless, it is also wrong since the original instruction
only redefines part of the register, and the value read by the new
instruction is wrong.
The problem was the LiveRangeEdit::allUsesAvailableAt() didn't
special-case OrigIdx == UseIdx and found the wrong SSA value.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@166068 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'tools')
0 files changed, 0 insertions, 0 deletions