[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