[PATCH] D89768: [Coroutine] Properly determine whether an alloca should live on the frame

Thu Nov 5 14:25:29 PST 2020

ben-clayton added a comment.

Hello,

A git bisect has identified this change as the likely candidate for a new set of asserts / crashes in SwiftShader <https://opensource.google/projects/swiftshader> when attempting to use the coroutine passes. This is having knock-on issues with internal Google projects.

When passing this IR <https://gist.github.com/ben-clayton/f86c02e7dc84357695b5e4d5ba65211b> to `./bin/opt crash.ir -coro-early -coro-split -coro-elide -S` with this change, we now get this crash <https://gist.github.com/ben-clayton/e2b383da11e71deec415b448dcea5ac9>.
Running the same command on the parent change does not crash, and behaves as expected.

I'd like to file a bug, but LLVM's Bugzilla is not letting me sign in, possibly due to spam restrictions. :-/

I'll do some investigation myself tomorrow, but any assistance here would be gratefully appreciated.

Many thanks,
Ben

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D89768