[PATCH] D67315: [SimplifyCFG] mergeConditionalStoreToAddress(): consider cost, not instruction count

[Roman Lebedev via Phabricator via llvm-commits]

lebedev.ri added a comment.

On Wed, Sep 18, 2019 at 9:02 PM James Molloy via Phabricator <reviews at reviews.llvm.org> wrote:

> jmolloy accepted this revision.
>  This revision is now accepted and ready to land.

Note: i **don't** //know// that this is the correct thing to do.
The old "cost" calculation seems, wrong?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D67315