diff options
| author | David Stenberg <david.stenberg@ericsson.com> | 2020-01-27 11:15:55 +0100 |
|---|---|---|
| committer | David Stenberg <david.stenberg@ericsson.com> | 2020-01-27 11:31:09 +0100 |
| commit | b46baa82fc3ec679cd6192b8ee04960c3e541ed1 (patch) | |
| tree | 85f584b249c54137ed5f1929953a5e39b29a48d8 /clang/lib/Frontend/CompilerInvocation.cpp | |
| parent | e77c149f0e4db4e51184f726bd3c5c83b17aa39a (diff) | |
| download | llvm-b46baa82fc3ec679cd6192b8ee04960c3e541ed1.zip llvm-b46baa82fc3ec679cd6192b8ee04960c3e541ed1.tar.gz llvm-b46baa82fc3ec679cd6192b8ee04960c3e541ed1.tar.bz2 | |
Don't separate imp/expl def handling for call site params
Summary:
Since D70431 the describeLoadedValue() hook takes a parameter register,
meaning that it can now be asked to describe any register. This means
that we can drop the difference between explicit and implicit defines
that we previously had in collectCallSiteParameters().
I have not found any case for any upstream targets where a parameter
register is only implicitly defined, and does not overlap with any
explicit defines. I don't know if such a case would even make sense. So
as far as I have tested, this patch should be a non-functional change.
However, this reduces the complexity of the code a bit, and it will
simplify the implementation of an upcoming patch which solves PR44118.
Reviewers: djtodoro, NikolaPrica, aprantl, vsk
Reviewed By: djtodoro, vsk
Subscribers: hiraditya, llvm-commits
Tags: #debug-info, #llvm
Differential Revision: https://reviews.llvm.org/D73167
Diffstat (limited to 'clang/lib/Frontend/CompilerInvocation.cpp')
0 files changed, 0 insertions, 0 deletions