diff options
| author | David Blaikie <dblaikie@gmail.com> | 2013-06-21 22:56:30 +0000 |
|---|---|---|
| committer | David Blaikie <dblaikie@gmail.com> | 2013-06-21 22:56:30 +0000 |
| commit | 1948910e3186d31bc0d213ecd0d7e87bb2c2760d (patch) | |
| tree | 72933564b70b854dfc26c146d92c8743e87da24c /lib/Target/ARM/ARMFastISel.cpp | |
| parent | b37e50b6601c0551b36d59b01fe80b9acd4703e5 (diff) | |
| download | external_llvm-1948910e3186d31bc0d213ecd0d7e87bb2c2760d.zip external_llvm-1948910e3186d31bc0d213ecd0d7e87bb2c2760d.tar.gz external_llvm-1948910e3186d31bc0d213ecd0d7e87bb2c2760d.tar.bz2 | |
DebugInfo: Don't lose unreferenced non-trivial by-value parameters
A FastISel optimization was causing us to emit no information for such
parameters & when they go missing we end up emitting a different
function type. By avoiding that shortcut we not only get types correct
(very important) but also location information (handy) - even if it's
only live at the start of a function & may be clobbered later.
Reviewed/discussion by Evan Cheng & Dan Gohman.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@184604 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Target/ARM/ARMFastISel.cpp')
| -rw-r--r-- | lib/Target/ARM/ARMFastISel.cpp | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/lib/Target/ARM/ARMFastISel.cpp b/lib/Target/ARM/ARMFastISel.cpp index 97dc63f..7f52749 100644 --- a/lib/Target/ARM/ARMFastISel.cpp +++ b/lib/Target/ARM/ARMFastISel.cpp @@ -3030,8 +3030,6 @@ bool ARMFastISel::FastLowerArguments() { Idx = 0; for (Function::const_arg_iterator I = F->arg_begin(), E = F->arg_end(); I != E; ++I, ++Idx) { - if (I->use_empty()) - continue; unsigned SrcReg = GPRArgRegs[Idx]; unsigned DstReg = FuncInfo.MF->addLiveIn(SrcReg, RC); // FIXME: Unfortunately it's necessary to emit a copy from the livein copy. |