[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