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

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 13 10:20:09 PDT 2021


spatel added reviewers: craig.topper, RKSimon.
spatel added a comment.

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?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D111596



More information about the llvm-commits mailing list