[cfe-commits] r89370 - in /cfe/trunk: lib/Sema/SemaExprCXX.cpp test/CodeGenCXX/new-with-default-arg.cpp

[Douglas Gregor]

On Nov 24, 2009, at 1:46 PM, Fariborz Jahanian wrote:

>
> On Nov 20, 2009, at 3:19 PM, Douglas Gregor wrote:
>
>>
>> On Nov 19, 2009, at 10:39 AM, Fariborz Jahanian wrote:
>>
>>> Author: fjahanian
>>> Date: Thu Nov 19 12:39:40 2009
>>> New Revision: 89370
>>>
>>> URL: http://llvm.org/viewvc/llvm-project?rev=89370&view=rev
>>> Log:
>>> Patch to implement new-operators with default args.
>>> Fixes pr5547.
>>>
>> We now have this code in three separate places:  
>> CompleteConstructorCall, ConvertArgumentsForCall, and here. Is  
>> there any way to unify this code?
>>
>> Also: I think this code will have to deal with conversions for  
>> arguments passed through "..." (or will have to pick up those  
>> conversions through the aforementioned unification).
>
> This is done now in r89781,  r89789 and r89802

Thanks!

	- Doug