[llvm] [DSE] Enable initializes improvement (PR #119116)
Mikael Holmén via llvm-commits
llvm-commits at lists.llvm.org
Wed Dec 11 01:52:47 PST 2024
mikaelholmen wrote:
Hi,
I see problems after this patch. I haven't run tests with ```-enable-dse-initializes-attr-improvement``` before but now I see that with
```opt -passes="dse" bbi-102254.ll -S -o - -enable-dse-initializes-attr-improvement```
the "memcpy" in the input program is simply removed. But that is needed because it writes to "@g", and without it "copyGto" won't copy the right data (to "@g" itself).
So is dse too aggressive here, or is the problem that there shouldn't be "initializes((0, 12))" on the "copyGto" input argument?
[bbi-102254.ll.gz](https://github.com/user-attachments/files/18093005/bbi-102254.ll.gz)
https://github.com/llvm/llvm-project/pull/119116
More information about the llvm-commits
mailing list