[PATCH] D154695: [Coroutines] Add an O(n) algorithm for computing the cross suspend point information.

witstorm via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 31 18:23:19 PDT 2023


witstorm95 added a comment.

In D154695#4549149 <https://reviews.llvm.org/D154695#4549149>, @MatzeB wrote:

> With current trunk I see invalid IR produced every 2nd or 3rd build. With this change reverted I was able to >10 builds in a row without invalid IR. I'd like to revert this.
>
> Unfortunately creating a reproducer is tricky given the nondeterminism so `llvm-reduce` doesn't work reliably. Would it be acceptable to revert this for the time being even without a reproducer?
>
> I don't think I grasped the full algorithm employed here. But give this is just a standard dataflow problem we are dealing with here, this seems too complicated to me. I can't shake the feeling that the code would be simpler with just visiting every block once in RPOT to deal with those artifical inputs from the task and then keep doing the normal worklist algorithm till the fixpoint (= the code before this change).

So the problem arose when you applied this patch ? If it is, then it's a problem with this patch, and I need to find out why. Could you provide more info about this?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D154695



More information about the llvm-commits mailing list