[PATCH] D104597: [SimplifyCFG] Tail-merging all blocks with `ret` terminator

[Dave Green via Phabricator via llvm-commits]

dmgreen added a comment.

In D104597#2837100 <https://reviews.llvm.org/D104597#2837100>, @rnk wrote:

> I'm pretty sure Clang generates a single return block, so this change alone probably won't affect any C/C++ benchmarks. Most of the affected tests are manually written IR, where LLVM developers tend to use multiple ret terminators for simplicity.
>
> I believe Clang also uses a single `resume` block, but I could be wrong.

Ah. That would explain the performance being flat. Codesize is different enough to suggest something is changing, but it's not enough of a difference to worry about.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D104597