[PATCH] D111596: [SelectionDAG] Bail out of mergeTruncStores when not optimizing

[Arthur Eubanks via Phabricator via llvm-commits]

aeubanks added a comment.

In D111596#3062068 <https://reviews.llvm.org/D111596#3062068>, @spatel wrote:

> I see similar checks sprinkled around DAGCombiner, so I guess this is ok too. Not sure if it is possible to do this at a higher level and remove those checks though.
>
> But shouldn't this change the codegen for a reduced example of https://llvm.org/PR51827 ? Or is there some other `optnone` check that hides this one?

This patch doesn't affect the codegen of the attached IR, but speeds it up a lot. It's probably bailing somewhere in `mergeTruncStores()` for most inputs.

F19628751: b.ll <https://reviews.llvm.org/F19628751>


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D111596