But it's completely empty, no?
const Type *llvm::ConvertibleToGEP(const Type *Ty, Value *OffsetVal,
std::vector<Value*> &Indices,
const TargetData &TD,
BasicBlock::iterator *BI) {
return 0;
}
in lib/Transforms/TransformInternals.cpp, how can this be?
Naftali
On Tue, 26 Jul 2005, Chris Lattner wrote:
> On Tue, 26 Jul 2005, Naftali Schwartz wrote:
>> It seems like general dependence analysis in LLVM should require that
this
>> function be, well, functional. Is it simply a placeholder waiting for
>> someone to come and fill in the details? It also appears that some
other
>> things besides dependence analysis depend on it as well...anyone
working on
>> this?
>
> I don't understand what you mean. This function is just a little
utility
> function used by the "-raise" pass.
>
> -Chris
>
> --
> http://nondot.org/sabre/
> http://llvm.org/
>
> _______________________________________________
> LLVM Developers mailing list
> LLVMdev at cs.uiuc.edu http://llvm.cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev
>
>
On Tue, 26 Jul 2005, Naftali Schwartz wrote:> But it's completely empty, no? > > const Type *llvm::ConvertibleToGEP(const Type *Ty, Value *OffsetVal, > std::vector<Value*> &Indices, > const TargetData &TD, > BasicBlock::iterator *BI) { > return 0; > } > > in lib/Transforms/TransformInternals.cpp, how can this be?Huh, interesting. I'll remove it, thanks for pointing it out! -Chris> On Tue, 26 Jul 2005, Chris Lattner wrote: > >> On Tue, 26 Jul 2005, Naftali Schwartz wrote: >>> It seems like general dependence analysis in LLVM should require that this >>> function be, well, functional. Is it simply a placeholder waiting for >>> someone to come and fill in the details? It also appears that some other >>> things besides dependence analysis depend on it as well...anyone working >>> on this? >> >> I don't understand what you mean. This function is just a little utility >> function used by the "-raise" pass. >> >> -Chris >> >> -- >> http://nondot.org/sabre/ >> http://llvm.org/ >> >> _______________________________________________ >> LLVM Developers mailing list >> LLVMdev at cs.uiuc.edu http://llvm.cs.uiuc.edu >> http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev >> >> > > _______________________________________________ > LLVM Developers mailing list > LLVMdev at cs.uiuc.edu http://llvm.cs.uiuc.edu > http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev >-Chris -- http://nondot.org/sabre/ http://llvm.org/
I'm sorry, it had seemed to me that the documented functionality: // ConvertibleToGEP - This function returns true if the specified value V is // a valid index into a pointer of type Ty. If it is valid, Idx is filled in // with the values that would be appropriate to make this a getelementptr // instruction. The type returned is the root type that the GEP would point to would be quite useful (and in particular for dependence analysis). Are you going to remove it because it's functionality is already present elsewhere or is simply not needed anymore? Naftali On Tue, 26 Jul 2005, Chris Lattner wrote:> On Tue, 26 Jul 2005, Naftali Schwartz wrote: >> But it's completely empty, no? >> >> const Type *llvm::ConvertibleToGEP(const Type *Ty, Value *OffsetVal, >> std::vector<Value*> &Indices, >> const TargetData &TD, >> BasicBlock::iterator *BI) { >> return 0; >> } >> >> in lib/Transforms/TransformInternals.cpp, how can this be? > > Huh, interesting. I'll remove it, thanks for pointing it out! > > -Chris > >> On Tue, 26 Jul 2005, Chris Lattner wrote: >> >>> On Tue, 26 Jul 2005, Naftali Schwartz wrote: >>>> It seems like general dependence analysis in LLVM should require that >>>> this function be, well, functional. Is it simply a placeholder waiting >>>> for someone to come and fill in the details? It also appears that some >>>> other things besides dependence analysis depend on it as well...anyone >>>> working on this? >>> >>> I don't understand what you mean. This function is just a little utility >>> function used by the "-raise" pass. >>> >>> -Chris >>> >>> -- >>> http://nondot.org/sabre/ >>> http://llvm.org/ >>> >>> _______________________________________________ >>> LLVM Developers mailing list >>> LLVMdev at cs.uiuc.edu http://llvm.cs.uiuc.edu >>> http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev >>> >>> >> >> _______________________________________________ >> LLVM Developers mailing list >> LLVMdev at cs.uiuc.edu http://llvm.cs.uiuc.edu >> http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev >> > > -Chris > > -- > http://nondot.org/sabre/ > http://llvm.org/ > > _______________________________________________ > LLVM Developers mailing list > LLVMdev at cs.uiuc.edu http://llvm.cs.uiuc.edu > http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev > >