diff options
| author | jeanPerier <jperier@nvidia.com> | 2024-10-23 12:33:34 +0200 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2024-10-23 12:33:34 +0200 |
| commit | 03cef62fe53d9927865f2acbde438fc35377f16e (patch) | |
| tree | bc840de391d31b27736f353229e0b1a57e064665 /llvm/lib/CodeGen/CodeGen.cpp | |
| parent | d89c1dbaf5f60c734f5168e8c778ba250d7d2d5e (diff) | |
| download | llvm-03cef62fe53d9927865f2acbde438fc35377f16e.zip llvm-03cef62fe53d9927865f2acbde438fc35377f16e.tar.gz llvm-03cef62fe53d9927865f2acbde438fc35377f16e.tar.bz2 | |
[flang][NFC] turn (h)fir.declare side effect into debug ressource alloca (#113321)
See https://reviews.llvm.org/D157626 for the rational of declare having
side effects.
The write effect is to scary for passes that look for read/write effects
without caring about the resource affected. I know Slava asked for it,
but I think the creation of the `DebuggingResource` was enough and that
a write is too much. The alloca effect is sufficient to prevent DCE to
remove it, which is all we care about currently.
This currently is flag as a reason for creating LHS temporary in
assignment to vector subscripted entity with array constructor.
There is a lot of read/write side effect analysis in the
"lower-hlfir-ordered-assignments" pass, and I feel like we will just
keep adding weird "debug ressource" bypassing here and there with these
side effects.
Diffstat (limited to 'llvm/lib/CodeGen/CodeGen.cpp')
0 files changed, 0 insertions, 0 deletions