diff options
| author | Sanjay Patel <spatel@rotateright.com> | 2022-12-02 09:18:11 -0500 |
|---|---|---|
| committer | Sanjay Patel <spatel@rotateright.com> | 2022-12-02 10:08:19 -0500 |
| commit | 0037e21f28adbeb9f8cac30016521d543561645e (patch) | |
| tree | 525e5dfefca0addefb623adc784d47a245f1de22 /flang/lib/Frontend/CompilerInvocation.cpp | |
| parent | 5ce3273ebf5be4df72b3167698336af5db80ea48 (diff) | |
| download | llvm-0037e21f28adbeb9f8cac30016521d543561645e.zip llvm-0037e21f28adbeb9f8cac30016521d543561645e.tar.gz llvm-0037e21f28adbeb9f8cac30016521d543561645e.tar.bz2 | |
[SDAG] bail out of mergeTruncStores() if there's any other use in the chain
This fixes the miscompile in issue #58883.
The test demonstrates that we gave up on store merging in that example.
This change should be strictly safe (just adds another clause
to avoid the transform), and it does not prohibit any existing
valid optimizations based on regression tests. I want to believe
that it's also a sufficient fix (possibly overkill), but I'm not
sure how to prove that.
Differential Revision: https://reviews.llvm.org/D137791
Diffstat (limited to 'flang/lib/Frontend/CompilerInvocation.cpp')
0 files changed, 0 insertions, 0 deletions