[PATCH] D115808: [DAGCombiner] Avoid combining adjacent stores at -O0 to improve debug experience

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 20 08:02:50 PST 2021


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

I agree that it would be better to have a higher-level fix for the problem and made a similar comment here:
D111596 <https://reviews.llvm.org/D111596>

But it doesn't seem like there's enough motivation to overcome whatever the underlying bugs are, so we have a bunch of these 'optnone' checks around DAGCombiner now.

For reference, the original bug that mentioned disabling the whole combiner was:
https://llvm.org/PR22346

With more discussion here:
https://lists.llvm.org/pipermail/llvm-dev/2016-May/099686.html


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D115808



More information about the llvm-commits mailing list