diff options
author | Johannes Doerfert <johannes@jdoerfert.de> | 2022-02-07 23:37:02 -0600 |
---|---|---|
committer | Johannes Doerfert <johannes@jdoerfert.de> | 2022-02-10 13:52:24 -0600 |
commit | d1387a26a54228da8cc275bba5f428b16676bc61 (patch) | |
tree | 18d036e71da7d5ad48787252e40046ef00308565 /llvm/lib/ProfileData/Coverage/CoverageMappingReader.cpp | |
parent | e39b41931264d9f774dbe151bb64f4b579cf1ee0 (diff) | |
download | llvm-d1387a26a54228da8cc275bba5f428b16676bc61.zip llvm-d1387a26a54228da8cc275bba5f428b16676bc61.tar.gz llvm-d1387a26a54228da8cc275bba5f428b16676bc61.tar.bz2 |
[Attributor][FIX] Reachability needs to account for readonly callees
The oversight caused us to ignore call sites that are effectively dead
when we computed reachability (or more precise the call edges of a
function). The problem is that loads in the readonly callee might depend
on stores prior to the callee. If we do not track the call edge we
mistakenly assumed the store before the call cannot reach the load.
The problem is nicely visible in:
`llvm/test/Transforms/Attributor/ArgumentPromotion/basictest.ll`
Caused by D118673.
Fixes https://github.com/llvm/llvm-project/issues/53726
Diffstat (limited to 'llvm/lib/ProfileData/Coverage/CoverageMappingReader.cpp')
0 files changed, 0 insertions, 0 deletions