Re: r325291 - [Coroutines] Use allocator overload when available

2018-02-16 Thread Brian Gesiak via cfe-commits
This should be fixed in https://reviews.llvm.org/rL325342. Thanks for your help! - Brian On Fri, Feb 16, 2018 at 9:07 AM, Brian Gesiak wrote: > Ah I'm sorry, I've reproduced this with a `-triple armeb-none-eabi` > argument. Douglas, do you mind if I fix forward? > > - Brian > > On Fri, Feb 16,

Re: r325291 - [Coroutines] Use allocator overload when available

2018-02-16 Thread Brian Gesiak via cfe-commits
Ah I'm sorry, I've reproduced this with a `-triple armeb-none-eabi` argument. Douglas, do you mind if I fix forward? - Brian On Fri, Feb 16, 2018 at 8:55 AM, Brian Gesiak wrote: > Hi Douglas, thanks for letting me know. I'm looking into this now. > > At first these error messages made me think

Re: r325291 - [Coroutines] Use allocator overload when available

2018-02-16 Thread Brian Gesiak via cfe-commits
Hi Douglas, thanks for letting me know. I'm looking into this now. At first these error messages made me think https://reviews.llvm.org/rL325291 was merged in, but not https://reviews.llvm.org/rL325288. But I guess that's not possible. Is there a way to find out the host triple used on this build

RE: r325291 - [Coroutines] Use allocator overload when available

2018-02-16 Thread via cfe-commits
Hi Brian, Your change is causing a test failure on the clang-cmake-armv7-a15 bot in the test Clang::SemaCXX/coroutines.cpp. http://lab.llvm.org:8011/builders/clang-cmake-armv7-a15/builds/15742 error: 'error' diagnostics seen but not expected: File /home/buildslave/buildslave/clang-cmake-arm