diff options
| author | Michael Kuperstein <mkuper@google.com> | 2016-07-15 22:31:14 +0000 |
|---|---|---|
| committer | Michael Kuperstein <mkuper@google.com> | 2016-07-15 22:31:14 +0000 |
| commit | be2e3f5ce55e5563d8e21fcd707bd563ddda77ee (patch) | |
| tree | ec24883063c9e38d43e763442960e6780b8c39b5 /clang/lib/Frontend/CreateInvocationFromCommandLine.cpp | |
| parent | cfb51f54ba8033a507a985763f0e7fc1b9b6d69f (diff) | |
| download | llvm-be2e3f5ce55e5563d8e21fcd707bd563ddda77ee.zip llvm-be2e3f5ce55e5563d8e21fcd707bd563ddda77ee.tar.gz llvm-be2e3f5ce55e5563d8e21fcd707bd563ddda77ee.tar.bz2 | |
ExpandPostRAPseudos should transfer implicit uses, not only implicit defs
Previously, we would expand:
%BL<def> = COPY %DL<kill>, %EBX<imp-use,kill>, %EBX<imp-def>
Into:
%BL<def> = MOV8rr %DL<kill>, %EBX<imp-def>
Dropping the imp-use on the floor.
That confused CriticalAntiDepBreaker, which (correctly) assumes that if an
instruction defs but doesn't use a register, that register is dead immediately
before the instruction - while in this case, the high lanes of EBX can be very
much alive.
This fixes PR28560.
Differential Revision: https://reviews.llvm.org/D22425
llvm-svn: 275634
Diffstat (limited to 'clang/lib/Frontend/CreateInvocationFromCommandLine.cpp')
0 files changed, 0 insertions, 0 deletions