[PATCH] D20262: [DSE]Split memset when the memset is small enough to be lowered to stores
Jun Bum Lim via llvm-commits
llvm-commits at lists.llvm.org
Tue May 24 09:07:34 PDT 2016
junbuml added a comment.
> These tests don't seem to give me a lot of information about what you expect to happen and when. Can you modify them to actually test that stores get eliminated (because if they are not, this whole thing seems speculative enough that it probably should be value profile guided in some fashion)
Thanks Daniel for your quick feedback. I will change the tests to check final stores impacted by this patch in DSE.
http://reviews.llvm.org/D20262
More information about the llvm-commits
mailing list