llvm dev - May 2008 - [LLVMdev] Duplicate wrapper function in LLVM-C.

Hi (Gordon).

There appears to be two wrapper functions for
DerivedType::refineAbstractTypeTo() in llvm-c (Core.h, Core.cpp):

void LLVMRefineAbstractType(LLVMTypeRef AbstractType, LLVMTypeRef ConcreteType){
  DerivedType *Ty = unwrap<DerivedType>(AbstractType);
  Ty->refineAbstractTypeTo(unwrap(ConcreteType));
}

void LLVMRefineType(LLVMTypeRef AbstractTy, LLVMTypeRef ConcreteTy) {
 
unwrap<DerivedType>(AbstractTy)->refineAbstractTypeTo(unwrap(ConcreteTy));
}

The attached patch removes the first one.

Regards,
-Mahadevan.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: rm_dup_wrapper_fn.patch
Type: text/x-diff
Size: 1143 bytes
Desc: not available
URL:
<http://lists.llvm.org/pipermail/llvm-dev/attachments/20080518/0fcad198/attachment.patch>

Thanks, applied here:

http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20080519/062600.html

— Gordon


On 2008-05-18, at 13:20, Mahadevan R wrote:
> Hi (Gordon).
>
> There appears to be two wrapper functions for
> DerivedType::refineAbstractTypeTo() in llvm-c (Core.h, Core.cpp):
>
> void LLVMRefineAbstractType(LLVMTypeRef AbstractType, LLVMTypeRef  
> ConcreteType){
>  DerivedType *Ty = unwrap<DerivedType>(AbstractType);
>  Ty->refineAbstractTypeTo(unwrap(ConcreteType));
> }
>
> void LLVMRefineType(LLVMTypeRef AbstractTy, LLVMTypeRef ConcreteTy) {
>  unwrap<DerivedType>(AbstractTy)- 
> >refineAbstractTypeTo(unwrap(ConcreteTy));
> }
>
> The attached patch removes the first one.