mirror of
https://github.com/c64scene-ar/llvm-6502.git
synced 2025-07-07 08:24:27 +00:00
[opaque pointer type] Avoid using PointerType::getElementType for a few cases of CallInst
Calls to llvm::Value::mutateType are becoming extra-sensitive now that instructions have extra type information that will not be derived from operands or result type (alloca, gep, load, call/invoke, etc... ). The special-handling for mutateType will get more complicated as this work continues - it might be worth making mutateType virtual & pushing the complexity down into the classes that need special handling. But with only two significant uses of mutateType (vectorization and linking) this seems OK for now. Totally open to ideas/suggestions/improvements, of course. With this, and a bunch of exceptions, we can roundtrip an indirect call site through bitcode and IR. (a direct call site is actually trickier... I haven't figured out how to deal with the IR deserializer's lazy construction of Function/GlobalVariable decl's based on the type of the entity which means looking through the "pointer to T" type referring to the global) git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@235458 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
@ -1284,14 +1284,26 @@ public:
|
||||
///
|
||||
class CallInst : public Instruction {
|
||||
AttributeSet AttributeList; ///< parameter attributes for call
|
||||
FunctionType *FTy;
|
||||
CallInst(const CallInst &CI);
|
||||
void init(Value *Func, ArrayRef<Value *> Args, const Twine &NameStr);
|
||||
void init(Value *Func, ArrayRef<Value *> Args, const Twine &NameStr) {
|
||||
init(cast<FunctionType>(
|
||||
cast<PointerType>(Func->getType())->getElementType()),
|
||||
Func, Args, NameStr);
|
||||
}
|
||||
void init(FunctionType *FTy, Value *Func, ArrayRef<Value *> Args,
|
||||
const Twine &NameStr);
|
||||
void init(Value *Func, const Twine &NameStr);
|
||||
|
||||
/// Construct a CallInst given a range of arguments.
|
||||
/// \brief Construct a CallInst from a range of arguments
|
||||
inline CallInst(Value *Func, ArrayRef<Value *> Args,
|
||||
inline CallInst(FunctionType *Ty, Value *Func, ArrayRef<Value *> Args,
|
||||
const Twine &NameStr, Instruction *InsertBefore);
|
||||
inline CallInst(Value *Func, ArrayRef<Value *> Args, const Twine &NameStr,
|
||||
Instruction *InsertBefore)
|
||||
: CallInst(cast<FunctionType>(
|
||||
cast<PointerType>(Func->getType())->getElementType()),
|
||||
Func, Args, NameStr, InsertBefore) {}
|
||||
|
||||
/// Construct a CallInst given a range of arguments.
|
||||
/// \brief Construct a CallInst from a range of arguments
|
||||
@ -1308,8 +1320,15 @@ public:
|
||||
ArrayRef<Value *> Args,
|
||||
const Twine &NameStr = "",
|
||||
Instruction *InsertBefore = nullptr) {
|
||||
return new(unsigned(Args.size() + 1))
|
||||
CallInst(Func, Args, NameStr, InsertBefore);
|
||||
return Create(cast<FunctionType>(
|
||||
cast<PointerType>(Func->getType())->getElementType()),
|
||||
Func, Args, NameStr, InsertBefore);
|
||||
}
|
||||
static CallInst *Create(FunctionType *Ty, Value *Func, ArrayRef<Value *> Args,
|
||||
const Twine &NameStr = "",
|
||||
Instruction *InsertBefore = nullptr) {
|
||||
return new (unsigned(Args.size() + 1))
|
||||
CallInst(Ty, Func, Args, NameStr, InsertBefore);
|
||||
}
|
||||
static CallInst *Create(Value *Func,
|
||||
ArrayRef<Value *> Args,
|
||||
@ -1347,9 +1366,11 @@ public:
|
||||
|
||||
~CallInst() override;
|
||||
|
||||
FunctionType *getFunctionType() const {
|
||||
return cast<FunctionType>(
|
||||
cast<PointerType>(getCalledValue()->getType())->getElementType());
|
||||
FunctionType *getFunctionType() const { return FTy; }
|
||||
|
||||
void mutateFunctionType(FunctionType *FTy) {
|
||||
mutateType(FTy->getReturnType());
|
||||
this->FTy = FTy;
|
||||
}
|
||||
|
||||
// Note that 'musttail' implies 'tail'.
|
||||
@ -1533,6 +1554,14 @@ public:
|
||||
|
||||
/// setCalledFunction - Set the function called.
|
||||
void setCalledFunction(Value* Fn) {
|
||||
setCalledFunction(
|
||||
cast<FunctionType>(cast<PointerType>(Fn->getType())->getElementType()),
|
||||
Fn);
|
||||
}
|
||||
void setCalledFunction(FunctionType *FTy, Value *Fn) {
|
||||
this->FTy = FTy;
|
||||
assert(FTy == cast<FunctionType>(
|
||||
cast<PointerType>(Fn->getType())->getElementType()));
|
||||
Op<-1>() = Fn;
|
||||
}
|
||||
|
||||
@ -1573,14 +1602,12 @@ CallInst::CallInst(Value *Func, ArrayRef<Value *> Args,
|
||||
init(Func, Args, NameStr);
|
||||
}
|
||||
|
||||
CallInst::CallInst(Value *Func, ArrayRef<Value *> Args,
|
||||
CallInst::CallInst(FunctionType *Ty, Value *Func, ArrayRef<Value *> Args,
|
||||
const Twine &NameStr, Instruction *InsertBefore)
|
||||
: Instruction(cast<FunctionType>(cast<PointerType>(Func->getType())
|
||||
->getElementType())->getReturnType(),
|
||||
Instruction::Call,
|
||||
OperandTraits<CallInst>::op_end(this) - (Args.size() + 1),
|
||||
unsigned(Args.size() + 1), InsertBefore) {
|
||||
init(Func, Args, NameStr);
|
||||
: Instruction(Ty->getReturnType(), Instruction::Call,
|
||||
OperandTraits<CallInst>::op_end(this) - (Args.size() + 1),
|
||||
unsigned(Args.size() + 1), InsertBefore) {
|
||||
init(Ty, Func, Args, NameStr);
|
||||
}
|
||||
|
||||
|
||||
@ -3034,6 +3061,7 @@ DEFINE_TRANSPARENT_OPERAND_ACCESSORS(IndirectBrInst, Value)
|
||||
///
|
||||
class InvokeInst : public TerminatorInst {
|
||||
AttributeSet AttributeList;
|
||||
FunctionType *FTy;
|
||||
InvokeInst(const InvokeInst &BI);
|
||||
void init(Value *Func, BasicBlock *IfNormal, BasicBlock *IfException,
|
||||
ArrayRef<Value *> Args, const Twine &NameStr);
|
||||
@ -3074,6 +3102,13 @@ public:
|
||||
/// Provide fast operand accessors
|
||||
DECLARE_TRANSPARENT_OPERAND_ACCESSORS(Value);
|
||||
|
||||
FunctionType *getFunctionType() const { return FTy; }
|
||||
|
||||
void mutateFunctionType(FunctionType *FTy) {
|
||||
mutateType(FTy->getReturnType());
|
||||
this->FTy = FTy;
|
||||
}
|
||||
|
||||
/// getNumArgOperands - Return the number of invoke arguments.
|
||||
///
|
||||
unsigned getNumArgOperands() const { return getNumOperands() - 3; }
|
||||
@ -3223,6 +3258,14 @@ public:
|
||||
|
||||
/// setCalledFunction - Set the function called.
|
||||
void setCalledFunction(Value* Fn) {
|
||||
setCalledFunction(
|
||||
cast<FunctionType>(cast<PointerType>(Fn->getType())->getElementType()),
|
||||
Fn);
|
||||
}
|
||||
void setCalledFunction(FunctionType *FTy, Value *Fn) {
|
||||
this->FTy = FTy;
|
||||
assert(FTy == cast<FunctionType>(
|
||||
cast<PointerType>(Fn->getType())->getElementType()));
|
||||
Op<-3>() = Fn;
|
||||
}
|
||||
|
||||
|
Reference in New Issue
Block a user