Chris Lattner wrote:
> On Jan 25, 2009, at 11:43 AM, Sebastian Redl wrote:
>> URL: http://llvm.org/viewvc/llvm-project?rev=62971&view=rev
>> Log:
>> Implement implicit conversions for pointers-to-member.
>
>> +      bool DerivationOkay = IsDerivedFrom(ToClass, FromClass, Paths);
>> +      assert(DerivationOkay &&
>> +             "Should not have been called if derivation isn't OK.");
>> +      if (!DerivationOkay)
>> +        return true;
>
> This 'if' (and others like it) are dead code, please remove it, the
> assertion should be enough.
The problem with removing it is that it makes the variable unused in
release builds. Do we have some cross-compiler way of suppressing
resulting warnings?

Sebastian
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to