[PATCH] D84763: DomTree: Make PostDomTree immune to block successors swap

Yevgeny Rouban via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Aug 2 23:42:03 PDT 2020


yrouban added a comment.

In D84763#2187490 <https://reviews.llvm.org/D84763#2187490>, @kuhar wrote:

> The code looks fine to me. Have you checked if compilation times are affected by this patch?

I have not checked compile time impact. I see 3 options:

1. Land the patch and hope that any regression will be caught by @nikic's  http://llvm-compile-time-tracker.com/
2. Ask @nikic  to check this particular patch before I land it
3. Ask @nikic to add a custom perf/ branch for me to test this patch on my own


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

https://reviews.llvm.org/D84763



More information about the llvm-commits mailing list