diff options
| author | Max Kazantsev <max.kazantsev@azul.com> | 2018-09-11 03:57:22 +0000 |
|---|---|---|
| committer | Max Kazantsev <max.kazantsev@azul.com> | 2018-09-11 03:57:22 +0000 |
| commit | e6413919cec6ac5e50d85103ef27a9b0c6ed57b0 (patch) | |
| tree | 07c00fa4142d1210ee7e5f3d40783b508d09a4d3 /llvm/lib/IR/Comdat.cpp | |
| parent | 1f52e38e8e06490c2efa38384efbd1302d21ffd9 (diff) | |
| download | bcm5719-llvm-e6413919cec6ac5e50d85103ef27a9b0c6ed57b0.tar.gz bcm5719-llvm-e6413919cec6ac5e50d85103ef27a9b0c6ed57b0.zip | |
[IndVars][NFC] Refactor to make modifications of Changed transparent
IndVarSimplify's design is somewhat odd in the way how it reports that
some transform has made a change. It has a `Changed` field which can
be set from within any function, which makes it hard to track whether or
not it was set properly after a transform was made. It leads to oversights
in setting this flag where needed, see example in PR38855.
This patch removes the `Changed` field, turns it into a local and unifies
the signatures of all relevant transform functions to return boolean value
which designates whether or not this transform has made a change.
Differential Revision: https://reviews.llvm.org/D51850
Reviewed By: skatkov
llvm-svn: 341893
Diffstat (limited to 'llvm/lib/IR/Comdat.cpp')
0 files changed, 0 insertions, 0 deletions
