<div dir="ltr">Hi,<div><br></div><div>I sent out a patch (<a href="https://reviews.llvm.org/D69121" target="_blank">D69121</a>)  removing the BasicBlockPass, BasicBlockManager and all their dependencies following the patch that cleaned up the instantiations of the BasicBlockPass <span style="background-color:transparent;font-variant-numeric:normal;font-variant-east-asian:normal;font-size:11pt;font-family:Arial;color:rgb(0,0,0);vertical-align:baseline;white-space:pre-wrap">(</span><a href="https://reviews.llvm.org/rL373254" target="_blank">rL373254</a><span style="font-variant-numeric:normal;font-variant-east-asian:normal;font-size:10pt;font-family:Arial;color:rgb(70,76,92);background-color:rgb(243,245,247);font-weight:700;vertical-align:baseline;white-space:pre-wrap">)</span>.</div><div><br></div><div>The motivation was that the BasicBlockManager was generally unused, untested, was potentially broken (<a href="https://bugs.llvm.org/show_bug.cgi?id=42264" target="_blank">PR42264</a>, <a href="https://bugs.llvm.org/show_bug.cgi?id=43429" target="_blank">PR43429</a>, <a href="https://bugs.llvm.org/show_bug.cgi?id=43336" target="_blank">PR43336</a>, <a href="https://bugs.llvm.org/show_bug.cgi?id=42246" target="_blank">PR42246</a>), and any BasicBlockPass could be updated to be a FunctionPass looping over all BBs.</div><div><br></div><div>If you have comments or concerns about this cleanup please let me know.</div><div><br></div><div>Best,</div><div>Alina</div></div>