diff options
| author | Vedant Kumar <vsk@apple.com> | 2018-12-15 00:03:33 +0000 |
|---|---|---|
| committer | Vedant Kumar <vsk@apple.com> | 2018-12-15 00:03:33 +0000 |
| commit | 9d1827331f3f9582fa2ed05913ae4301f2604a19 (patch) | |
| tree | 0ff3fa53b9f20ffeff192d0f653cafb0674e0b12 /lldb/source/Symbol/ObjectFile.cpp | |
| parent | 6d74bd638a85cc17f3d5784e41d3d0162aad787c (diff) | |
| download | bcm5719-llvm-9d1827331f3f9582fa2ed05913ae4301f2604a19.tar.gz bcm5719-llvm-9d1827331f3f9582fa2ed05913ae4301f2604a19.zip | |
[Util] Refer to [s|z]exts of args when converting dbg.declares (fix PR35400)
When converting dbg.declares, if the described value is a [s|z]ext,
refer to the ext directly instead of referring to its operand.
This fixes a narrowing bug (the debugger got the sign of a variable
wrong, see llvm.org/PR35400).
The main reason to refer to the ext's operand was that an optimization
may remove the ext itself, leading to a dropped variable. Now that
InstCombine has been taught to use replaceAllDbgUsesWith (r336451), this
is less of a concern. Other passes can/should adopt this API as needed
to fix dropped variable bugs.
Differential Revision: https://reviews.llvm.org/D51813
llvm-svn: 349214
Diffstat (limited to 'lldb/source/Symbol/ObjectFile.cpp')
0 files changed, 0 insertions, 0 deletions
