[PATCH] D156835: CoroFrame: Rework SuspendCrossingInfo analysis

Matthias Braun via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 2 16:15:12 PDT 2023


MatzeB added a comment.

I finally understand what happened. Turns out the variant loop runtime of the Worklist algorithm is dominated by some exception handling block having N predecessors for an input of genX input of size N. We updated every of those N predecessors and each time re-visisted that block and merged the info of all the N predecessors. So not sure what to make of this... it's surely a bit of an artifact of those artifically big inputs, but yeah we happen to avoid this problem by just doing multiple rounds of RPO over the whole functions as in https://reviews.llvm.org/D156850 so let's go with that.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D156835



More information about the llvm-commits mailing list