[PATCH] D61314: [SCCP] Remove forcedconstant, go to overdefined instead

Tue Feb 11 07:32:34 PST 2020

fhahn added a comment.

In D61314#1868393 <https://reviews.llvm.org/D61314#1868393>, @efriedma wrote:

> LGTM
>
> I'm surprised this works so well, but I'm very happy to get rid of forcedconstant if there isn't a practical performance impact.

Thanks Eli! I'll collected the stats again and the impact seems very low, as indicated, but I'll make sure to keep an eye out for any performance regressions.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D61314