Re: r326766 - [ExprConstant] Look through ExprWithCleanups for `allocsize`

2018-03-06 Thread George Burgess IV via cfe-commits
Relanded in r326872. Thanks! On Tue, Mar 6, 2018 at 7:03 PM, Nico Weber wrote: > Apologies, I had to revert the change that touched alloc-size.cpp before > this change in r326862. After that revert, your new test here failed, and > since I didn't understand how to make it

Re: r326766 - [ExprConstant] Look through ExprWithCleanups for `allocsize`

2018-03-06 Thread Nico Weber via cfe-commits
Apologies, I had to revert the change that touched alloc-size.cpp before this change in r326862. After that revert, your new test here failed, and since I didn't understand how to make it passed, I reverted your change in 326869 too. It should hopefully be easy for you to reland it. On Tue, Mar

r326766 - [ExprConstant] Look through ExprWithCleanups for `allocsize`

2018-03-05 Thread George Burgess IV via cfe-commits
Author: gbiv Date: Mon Mar 5 23:42:36 2018 New Revision: 326766 URL: http://llvm.org/viewvc/llvm-project?rev=326766=rev Log: [ExprConstant] Look through ExprWithCleanups for `allocsize` Modified: cfe/trunk/lib/AST/ExprConstant.cpp cfe/trunk/test/CodeGenCXX/alloc-size.cpp Modified: