[PATCH] D105338: [InstCombine] Revert "Temporarily do not drop volatile stores before unreachable"

[Dávid Bolvanský via Phabricator via llvm-commits]

xbolva00 added a comment.

So maybe revert and provide a plan how to progress with it? No need for rush. Diag patch should be reviewed properly as well.

The aggressive style of commiting important patches without consensus is definitely a bad thing for the quality of LLVM. You can be aggressive in your LLVM forks, with you experimental builds, but not in the upstream.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D105338