[PATCH] D75846: [SCCP] Split up callsite handling, only propagate result on change (NFC)

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 12 08:08:41 PDT 2020


fhahn added a comment.

In D75846#1919428 <https://reviews.llvm.org/D75846#1919428>, @fhahn wrote:

> In D75846#1917389 <https://reviews.llvm.org/D75846#1917389>, @efriedma wrote:
>
> > This makes sense, I think.  No testcase because you can't test it on its own?
>
>
> Yes, this change should be NFC, as we should only skip redundant updates without constant ranges.


I've updated the title to reflect that. There are also no changes in the binaries when building MultiSource, SPEC2000 & SPEC2006 with and without the patch.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D75846





More information about the llvm-commits mailing list