diff options
| author | Teresa Johnson <tejohnson@google.com> | 2020-02-26 10:54:56 -0800 |
|---|---|---|
| committer | Teresa Johnson <tejohnson@google.com> | 2020-03-02 09:31:44 -0800 |
| commit | 873c0d0786dcf22f4af39f65df824917f70f2170 (patch) | |
| tree | 337821562238fd6c8e4a45c4b431f6c94e868c86 /clang/lib/Frontend/CompilerInvocation.cpp | |
| parent | 7a25bd1d1950f39935afecdebf3c59d2b3f4eae4 (diff) | |
| download | llvm-873c0d0786dcf22f4af39f65df824917f70f2170.zip llvm-873c0d0786dcf22f4af39f65df824917f70f2170.tar.gz llvm-873c0d0786dcf22f4af39f65df824917f70f2170.tar.bz2 | |
[ThinLTO/LowerTypeTests] Handle unpromoted local type ids
Summary:
Fixes an issue that cropped up after the changes in D73242 to delay
the lowering of type tests. LTT couldn't handle any type tests with
non-string type id (which happens for local vtables, which we try to
promote during the compile step but cannot always when there are no
exported symbols).
We can simply treat the same as having an Unknown resolution, which
delays their lowering, still allowing such type tests to be used in
subsequent optimization (e.g. planned usage during ICP). The final
lowering which simply removes these handles them fine.
Beefed up an existing ThinLTO test for such unpromoted type ids so that
the internal vtable isn't removed before lower type tests, which hides
the problem.
Reviewers: evgeny777, pcc
Subscribers: inglorion, hiraditya, steven_wu, dexonsmith, aganea, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D75201
Diffstat (limited to 'clang/lib/Frontend/CompilerInvocation.cpp')
0 files changed, 0 insertions, 0 deletions