search for: ccvalassign

...hing > like > > CCIfType<[f64], CCCustom> > > where CCCustom is a new entity that tells the calling convention > code to to let the target do something not easily representable > in the tablegen minilanguage. I am thinking that this requires two changes: add a flag to CCValAssign (take a bit from HTP) to indicate isCustom and a way to author an arbitrary CCAction by including the source directly in the TableGen mini-language. This latter change might want a generic change to the TableGen language. For example, the syntax might be like: class foo : CCCustomAction { code &lt...

...; >>> >>> where CCCustom is a new entity that tells the calling convention >>> code to to let the target do something not easily representable >>> in the tablegen minilanguage. >> >> I am thinking that this requires two changes: add a flag to >> CCValAssign (take a bit from HTP) to indicate isCustom and a way to >> author an arbitrary CCAction by including the source directly in the >> TableGen mini-language. This latter change might want a generic change >> to the TableGen language. For example, the syntax might be like: >> &g...

I find the lldb cannot display structure type information, such as "print ArgLocs.size()" as follows, -> 593 CCValAssign &VA = ArgLocs[i]; 594 EVT ValVT = VA.getValVT(); 595 ISD::ArgFlagsTy Flags = Ins[i].Flags; 596 bool IsRegLoc = VA.isRegLoc(); (lldb) print ArgLocs.size() error: call to a function 'llvm::SmallVectorTemplateCommon<llvm::CCValAssign>::size() const' that is no...

...ed? Are we running out of bits? - NeededStackSize = 4; - break; - case MVT::i64: - case MVT::f64: - if (firstGPR < 3) - NeededGPRs = 2; - else if (firstGPR == 3) { - NeededGPRs = 1; - NeededStackSize = 4; - } else - NeededStackSize = 8; + State.addLoc(CCValAssign::getCustomMem(ValNo, ValVT, + State.AllocateStack(4, 4), + MVT::i32, LocInfo)); + return true; // we handled it Your change isn't handling the "NeededStackSize = 8" case. ++ static const uns...

...location for Val, possibly > updating > +/// all args to reflect changes and indicates if it handled it. It > must set > +/// isCustom if it handles the arg and returns true. > +typedef bool CCCustomFn(unsigned &ValNo, MVT &ValVT, > + MVT &LocVT, CCValAssign::LocInfo &LocInfo, > + ISD::ArgFlagsTy &ArgFlags, CCState &State, > + bool &result); > > Is it necessary to return two bools (the second is returned by > reference in 'result')? I am confused about the semantics of...

...= 4; > - break; > - case MVT::i64: > - case MVT::f64: > - if (firstGPR < 3) > - NeededGPRs = 2; > - else if (firstGPR == 3) { > - NeededGPRs = 1; > - NeededStackSize = 4; > - } else > - NeededStackSize = 8; > + State.addLoc(CCValAssign::getCustomMem(ValNo, ValVT, > + State.AllocateStack(4, 4), > + MVT::i32, LocInfo)); > + return true; // we handled it > > Your change isn't handling the "NeededStackSize = 8" cas...

...IfType<[f64], CCCustom> >> >> where CCCustom is a new entity that tells the calling convention >> code to to let the target do something not easily representable >> in the tablegen minilanguage. > > I am thinking that this requires two changes: add a flag to > CCValAssign (take a bit from HTP) to indicate isCustom and a way to > author an arbitrary CCAction by including the source directly in the > TableGen mini-language. This latter change might want a generic change > to the TableGen language. For example, the syntax might be like: > > class foo : C...

...MVT::i64: >> - case MVT::f64: >> - if (firstGPR < 3) >> - NeededGPRs = 2; >> - else if (firstGPR == 3) { >> - NeededGPRs = 1; >> - NeededStackSize = 4; >> - } else >> - NeededStackSize = 8; >> + State.addLoc(CCValAssign::getCustomMem(ValNo, ValVT, >> + State.AllocateStack(4, 4), >> + MVT::i32, LocInfo)); >> + return true; // we handled it >> >> Your change isn't handling the "NeededSta...

...- This function assigns a location for Val, possibly updating +/// all args to reflect changes and indicates if it handled it. It must set +/// isCustom if it handles the arg and returns true. +typedef bool CCCustomFn(unsigned &ValNo, MVT &ValVT, + MVT &LocVT, CCValAssign::LocInfo &LocInfo, + ISD::ArgFlagsTy &ArgFlags, CCState &State, + bool &result); Is it necessary to return two bools (the second is returned by reference in 'result')? I am confused about the semantics of 'result'. Al...

...- This function assigns a location for Val, possibly updating +/// all args to reflect changes and indicates if it handled it. It must set +/// isCustom if it handles the arg and returns true. +typedef bool CCCustomFn(unsigned &ValNo, MVT &ValVT, + MVT &LocVT, CCValAssign::LocInfo &LocInfo, + ISD::ArgFlagsTy &ArgFlags, CCState &State, + bool &result); Is "result" what you refer to as "isCustom" in the comments? Sorry, I am still confused. You mean it could return true but set '...

...location for Val, possibly > updating > +/// all args to reflect changes and indicates if it handled it. It > must set > +/// isCustom if it handles the arg and returns true. > +typedef bool CCCustomFn(unsigned &ValNo, MVT &ValVT, > + MVT &LocVT, CCValAssign::LocInfo &LocInfo, > + ISD::ArgFlagsTy &ArgFlags, CCState &State, > + bool &result); > > Is "result" what you refer to as "isCustom" in the comments? > > Sorry, I am still confused. You mean it could...

...f64: >>> - if (firstGPR < 3) >>> - NeededGPRs = 2; >>> - else if (firstGPR == 3) { >>> - NeededGPRs = 1; >>> - NeededStackSize = 4; >>> - } else >>> - NeededStackSize = 8; >>> + State.addLoc(CCValAssign::getCustomMem(ValNo, ValVT, >>> + >>> State.AllocateStack(4, 4), >>> + MVT::i32, LocInfo)); >>> + return true; // we handled it >>> >>> Your change isn...

...eflect changes and indicates if it handled it. It >>>>> must set >>>>> +/// isCustom if it handles the arg and returns true. >>>>> +typedef bool CCCustomFn(unsigned &ValNo, MVT &ValVT, >>>>> + MVT &LocVT, CCValAssign::LocInfo &LocInfo, >>>>> + ISD::ArgFlagsTy &ArgFlags, CCState &State, >>>>> + bool &result); >>>>> >>>>> Is "result" what you refer to as "isCustom" in the c...

...nd indicates if it handled it. It >>>>>> must set >>>>>> +/// isCustom if it handles the arg and returns true. >>>>>> +typedef bool CCCustomFn(unsigned &ValNo, MVT &ValVT, >>>>>> + MVT &LocVT, CCValAssign::LocInfo &LocInfo, >>>>>> + ISD::ArgFlagsTy &ArgFlags, CCState &State, >>>>>> + bool &result); >>>>>> >>>>>> Is "result" what you refer to as "isCusto...

...irstGPR < 3) >>>> -      NeededGPRs = 2; >>>> -    else if (firstGPR == 3) { >>>> -      NeededGPRs = 1; >>>> -      NeededStackSize = 4; >>>> -    } else >>>> -      NeededStackSize = 8; >>>> +      State.addLoc(CCValAssign::getCustomMem(ValNo, ValVT, >>>> + >>>> State.AllocateStack(4, 4), >>>> +                                             MVT::i32, LocInfo)); >>>> +    return true;  // we handled it >>>> >>>> Your change isn't handling the &q...

...>>> +/// all args to reflect changes and indicates if it handled it. It >>> must set >>> +/// isCustom if it handles the arg and returns true. >>> +typedef bool CCCustomFn(unsigned &ValNo, MVT &ValVT, >>> + MVT &LocVT, CCValAssign::LocInfo &LocInfo, >>> + ISD::ArgFlagsTy &ArgFlags, CCState &State, >>> + bool &result); >>> >>> Is "result" what you refer to as "isCustom" in the comments? >>> >>&g...

...ly >> updating >> +/// all args to reflect changes and indicates if it handled it. It >> must set >> +/// isCustom if it handles the arg and returns true. >> +typedef bool CCCustomFn(unsigned &ValNo, MVT &ValVT, >> + MVT &LocVT, CCValAssign::LocInfo &LocInfo, >> + ISD::ArgFlagsTy &ArgFlags, CCState &State, >> + bool &result); >> >> Is "result" what you refer to as "isCustom" in the comments? >> >> Sorry, I am still con...

...Rs = 2; >>>>>> - else if (firstGPR == 3) { >>>>>> - NeededGPRs = 1; >>>>>> - NeededStackSize = 4; >>>>>> - } else >>>>>> - NeededStackSize = 8; >>>>>> + State.addLoc(CCValAssign::getCustomMem(ValNo, ValVT, >>>>>> + >>>>>> State.AllocateStack(4, 4), >>>>>> + MVT::i32, >>>>>> LocInfo)); >>>>>> + return true; // we handled it >>>...

...>> -      NeededGPRs = 2; >>>>> -    else if (firstGPR == 3) { >>>>> -      NeededGPRs = 1; >>>>> -      NeededStackSize = 4; >>>>> -    } else >>>>> -      NeededStackSize = 8; >>>>> +      State.addLoc(CCValAssign::getCustomMem(ValNo, ValVT, >>>>> + >>>>> State.AllocateStack(4, 4), >>>>> +                                             MVT::i32, LocInfo)); >>>>> +    return true;  // we handled it >>>>> >>>>> Your change i...

...>>> - else if (firstGPR == 3) { >>>>>>> - NeededGPRs = 1; >>>>>>> - NeededStackSize = 4; >>>>>>> - } else >>>>>>> - NeededStackSize = 8; >>>>>>> + State.addLoc(CCValAssign::getCustomMem(ValNo, ValVT, >>>>>>> + >>>>>>> State.AllocateStack(4, 4), >>>>>>> + MVT::i32, >>>>>>> LocInfo)); >>>>>>> + return true; // we han...