[PATCH] D69121: [LegacyPassManager] Delete BasicBlockPass/Manager.

Eric Christopher via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 28 18:23:23 PDT 2019


echristo accepted this revision.
echristo added a comment.
This revision is now accepted and ready to land.

The straight removal and updates definitely make sense as-is. As far as I can tell the C API doesn't have support for BBP and we're fine deleting it even if so.

Things that should happen:

- We should absolutely put this as part of the release notes for the next release.

I can't think of anything else we might be missing here. Might want to send a message to llvm-dev to let people know it's happened in the near term.

Thanks!


Repository:
  rL LLVM

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

https://reviews.llvm.org/D69121





More information about the llvm-commits mailing list