[PATCH] D31584: [coroutines] Add support for allocation elision

2017-05-23 Thread Aleksey Shlyapnikov via Phabricator via cfe-commits
alekseyshl added a comment. This bot reports leaks in coro-alloc.cpp: http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux-bootstrap/builds/1401/steps/check-clang%20asan/logs/stdio. Please fix. Repository: rL LLVM https://reviews.llvm.org/D31584 __

[PATCH] D31584: [coroutines] Add support for allocation elision

2017-05-22 Thread Gor Nishanov via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL303596: [coroutines] Add support for allocation elision (authored by GorNishanov). Changed prior to commit: https://reviews.llvm.org/D31584?vs=93805&id=99838#toc Repository: rL LLVM https://reviews.

[PATCH] D31584: [coroutines] Add support for allocation elision

2017-04-16 Thread Gor Nishanov via Phabricator via cfe-commits
GorNishanov added a comment. It is a simple change. If there is no objections, I'll commit it tomorrow. https://reviews.llvm.org/D31584 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commit

[PATCH] D31584: [coroutines] Add support for allocation elision

2017-04-10 Thread Gor Nishanov via Phabricator via cfe-commits
GorNishanov added a comment. Gentle ping https://reviews.llvm.org/D31584 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D31584: [coroutines] Add support for allocation elision

2017-04-02 Thread Gor Nishanov via Phabricator via cfe-commits
GorNishanov created this revision. We wrap allocation code so that backend can elide it if necessary. llvm.coro.alloc intrinsic returns true, when allocation is needed and false otherwise. %NeedAlloc = call i1 @llvm.coro.alloc(token %2) br i1 %NeedAlloc, label %AllocBB, label %InitBB