[PATCH] D89917: Revert "SimplifyCFG: Clean up optforfuzzing implementation"

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 21 23:15:42 PDT 2020


lebedev.ri accepted this revision.
lebedev.ri added a comment.

Thank you for the patch!

In D89917#2346037 <https://reviews.llvm.org/D89917#2346037>, @arsenm wrote:

> I disagree with this change. SimplifyCFG should continue to pass in options for transform options as is done for the others. I don't think avoiding duplication between the new PM and code that will be deleted is worth it

I disagree here. Why is that beneficial, if we want to force-disable those transforms regardless of their previous configured status?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D89917



More information about the llvm-commits mailing list