[PATCH] D43000: [Coroutines] Don't move stores for allocator args

Brian Gesiak via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 14 08:20:33 PST 2018


modocache updated this revision to Diff 134236.
modocache added a comment.

Thanks for the review, @GorNishanov! I added something similar to your example of a store operand to the test case. Indeed, without the changes you proposed, the existence of the operand caused an invariant to be violated in LLVM. With your suggested changes, the assertion doesn't fire and the new test passes.


Repository:
  rL LLVM

https://reviews.llvm.org/D43000

Files:
  lib/Transforms/Coroutines/CoroSplit.cpp
  test/Transforms/Coroutines/coro-split-alloc.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D43000.134236.patch
Type: text/x-patch
Size: 3800 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180214/dea71b2e/attachment.bin>


More information about the llvm-commits mailing list