Recognize masked stores and eliminate dead ones.
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
Comment Actions
Could you also add a test that uses the MemorySSA DSE implementation (-enable-dse-memoryssa), which will become the default very soon.
Comment Actions
Committed in https://reviews.llvm.org/rG81ff2d30a900. Forgot to update the commit message. :(
Comment Actions
I disabled MSSA in this testcase so it won't fail when you switch (db7defd9bab7527ec1d0ed3fc62b379a9adf0971). I'll fix it in the meantime.
Comment Actions
Could you please prioritize figuring out why the test fails with DSE+MSSA? It's important to have this clarified before the flag flip.
Comment Actions
I suspect it needs adding support for masked_store to getLocForWriteEx. It fails as in it is a missed optimization, right?