[llvm-commits] [llvm] r171197 - /llvm/trunk/lib/Analysis/InstructionSimplify.cpp

[David Blaikie]

On Fri, Dec 28, 2012 at 9:04 AM, David Blaikie <dblaikie at gmail.com> wrote:
> On Fri, Dec 28, 2012 at 6:50 AM, Chandler Carruth <chandlerc at gmail.com> wrote:
>> Author: chandlerc
>> Date: Fri Dec 28 08:50:51 2012
>> New Revision: 171197
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=171197&view=rev
>> Log:
>> Nuke some dead code that snuck in some how. I thought I had already
>> deleted this, but apparantly not. Charmingly, Clang didn't warn on it
>> but GCC did.

Though, curiously, I couldn't reproduce, with a simple test case, the
GCC warning you mentioned.

>
> I'd hazard a guess that Clang's -Wunused-function only triggers on
> template specializations (since it's powered by "which symbols do we
> actually bother emitting into the object file" basically). Filed PR
> 14733 to track this.
>
>>
>> Modified:
>>     llvm/trunk/lib/Analysis/InstructionSimplify.cpp
>>
>> Modified: llvm/trunk/lib/Analysis/InstructionSimplify.cpp
>> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Analysis/InstructionSimplify.cpp?rev=171197&r1=171196&r2=171197&view=diff
>> ==============================================================================
>> --- llvm/trunk/lib/Analysis/InstructionSimplify.cpp (original)
>> +++ llvm/trunk/lib/Analysis/InstructionSimplify.cpp Fri Dec 28 08:50:51 2012
>> @@ -2870,11 +2870,6 @@
>>  }
>>
>>  template <typename IterTy>
>> -static Value *SimplifyIntrinsic(Intrinsic::ID IID, IterTy ArgBegin, IterTy ArgEnd,
>> -                                const Query &Q, unsigned MaxRecurse) {
>> -}
>> -
>> -template <typename IterTy>
>>  static Value *SimplifyCall(Value *V, IterTy ArgBegin, IterTy ArgEnd,
>>                             const Query &Q, unsigned MaxRecurse) {
>>    Type *Ty = V->getType();
>>
>>
>> _______________________________________________
>> llvm-commits mailing list
>> llvm-commits at cs.uiuc.edu
>> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits