diff options
| author | Florian Hahn <flo@fhahn.com> | 2019-12-04 17:26:29 +0000 |
|---|---|---|
| committer | Florian Hahn <flo@fhahn.com> | 2019-12-04 17:46:01 +0000 |
| commit | e8a5c17211339a747d2bf5b0df708c8392121e6d (patch) | |
| tree | da014a5c59fe8a6b106a1b01b05284e67d5869ae /llvm/lib/ProfileData/Coverage/CoverageMappingWriter.cpp | |
| parent | a249551bb232414b68525d40cd7bae617293c67b (diff) | |
| download | bcm5719-llvm-e8a5c17211339a747d2bf5b0df708c8392121e6d.tar.gz bcm5719-llvm-e8a5c17211339a747d2bf5b0df708c8392121e6d.zip | |
[LoopInterchange] Improve inner exit loop safety checks.
The PHI node checks for inner loop exits are too permissive currently.
As indicated by an existing comment, we should only allow LCSSA PHI
nodes that are part of reductions or are only used outside of the loop
nest. We ensure this by checking the users of the LCSSA PHIs.
Specifically, it is not safe to use an exiting value from the inner loop in the latch of the outer
loop.
It also moves the inner loop exit check before the outer loop exit
check.
Fixes PR43473.
Reviewers: efriedma, mcrosier
Reviewed By: efriedma
Differential Revision: https://reviews.llvm.org/D68144
Diffstat (limited to 'llvm/lib/ProfileData/Coverage/CoverageMappingWriter.cpp')
0 files changed, 0 insertions, 0 deletions
