Gently ping.
> On Aug 26, 2015, at 2:40 PM, Ettore Speziale <speziale.ett...@gmail.com>
> wrote:
>
> Forward to the right ML:
>
>>> Sorry about the extreme delay. This patch slipped through the cracks, and I
>>> only noticed it again when searching my email for enable_if. Committed in
>>> r245985! In the future, please feel free to continue pinging weekly!
>>
>> NP, thank you for committing the patch.
>>
>> Unfortunately it contains a little error in the case of no candidate has
>> been found. For instance consider the following test case:
>>
>> struct Incomplete;
>>
>> struct X {
>> void hidden_by_argument_conversion(Incomplete n, int m = 0)
>> __attribute((enable_if(m == 10, "chosen when 'm' is ten")));
>> };
>>
>> x.hidden_by_argument_conversion(10);
>>
>> I would expect to get an error about Incomplete, as the compiler cannot
>> understand how to convert 10 into an instance of Incomplete. However right
>> now the enable_if diagnostic is emitted, thus masking the more useful
>> message about Incomplete.
>>
>> The attached patch solved the problem by delaying the point where the
>> enable_if diagnostic is issued.
>>
>> Thanks,
>> Ettore Speziale
>
>
> <enable_if.diff>
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
