[PATCH] D138027: Clean up CFG on JumpThreading

Pedro Olsen Ferreira via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 19 05:31:08 PST 2022


arkangath added a comment.

In D138027#4004662 <https://reviews.llvm.org/D138027#4004662>, @nikic wrote:

> This doesn't really make sense to me. It deletes trivial cycles, but there may be dead cycles that are larger.

I don't understand what you mean.
Should the patch be rejected because it doesn't clean *everything* ?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D138027



More information about the llvm-commits mailing list