search for: valvt

...- 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 unsigned HiRegList[] = { ARM::...

...se 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. I believe it is. I'...

...64: >> - 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. >...

...PR < 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 "N...

...-      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&quo...

...- 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 >>>>>> >>>&g...

...gt;>>>> -    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 &quot...

...PR == 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 >>>>&gt...

...> 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); >>>>>> &g...

...>>>>> 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); >>>>> >>>>&g...

...> > +/// CCCustomFn - 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...

...d processing of the arg or not +/// CCCustomFn - 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 c...

...> > +/// CCCustomFn - 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 commen...

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 not present in the target error: Couldn't...

...gt; results: if the CC processing is finished and if it the func succeeded >> or failed: > > I like the second solution better. It seems rather cumbersome to embed > multi-line c++ code in td files. > > Evan >> >> >> typedef bool CCCustomFn(unsigned ValNo, MVT ValVT, >> MVT LocVT, CCValAssign::LocInfo LocInfo, >> ISD::ArgFlagsTy ArgFlags, CCState &State, >> bool &result); >> _______________________________________________ >> LLVM Developers mailing list >> LLVMdev at cs.u...

...on 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...

...stomFn - 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...

...d. But I do have a question: +/// CCCustomFn - 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...

...t;>> 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&q...

...t names a function to be called: CCIfType<[f64], CCCustom<"MyCustomLoweringFunc">> the function signature for such functions will have to return two results: if the CC processing is finished and if it the func succeeded or failed: typedef bool CCCustomFn(unsigned ValNo, MVT ValVT, MVT LocVT, CCValAssign::LocInfo LocInfo, ISD::ArgFlagsTy ArgFlags, CCState &State, bool &result);