[PATCH] D79391: [DSE] Remove noop stores in MSSA.

Alina Sbirlea via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 20 23:58:59 PDT 2020


asbirlea added a comment.

In D79391#2048050 <https://reviews.llvm.org/D79391#2048050>, @xbolva00 wrote:

> >> Compile Time: (unexpectedly improved but, that might just be noise)
>
> Regressed kinda a lot ?


Yup! 7%-15% is huge. I'm hoping this is due to the unnecessary processing I commented on, but won't know until tested with the changes. I'm pretty curios which part led to that big of a regression.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D79391/new/

https://reviews.llvm.org/D79391





More information about the llvm-commits mailing list