<html xmlns:v="urn:schemas-microsoft-com:vml" xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40">

<head>

<meta http-equiv="Content-Type" content="text/html; charset=iso-2022-jp">

<meta name="Generator" content="Microsoft Word 14 (filtered medium)">

<style><!--

/* Font Definitions */

@font-face

        {font-family:Calibri;

        panose-1:2 15 5 2 2 2 4 3 2 4;}

@font-face

        {font-family:Tahoma;

        panose-1:2 11 6 4 3 5 4 4 2 4;}

@font-face

        {font-family:"Meiryo UI";

        panose-1:2 11 6 4 3 5 4 4 2 4;}

@font-face

        {font-family:"MS PGothic";

        panose-1:2 11 6 0 7 2 5 8 2 4;}

@font-face

        {font-family:"\@MS PGothic";

        panose-1:2 11 6 0 7 2 5 8 2 4;}

@font-face

        {font-family:"\@Meiryo UI";

        panose-1:2 11 6 4 3 5 4 4 2 4;}

/* Style Definitions */

p.MsoNormal, li.MsoNormal, div.MsoNormal

        {margin:0cm;

        margin-bottom:.0001pt;

        font-size:11.0pt;

        font-family:"Calibri","sans-serif";}

a:link, span.MsoHyperlink

        {mso-style-priority:99;

        color:blue;

        text-decoration:underline;}

a:visited, span.MsoHyperlinkFollowed

        {mso-style-priority:99;

        color:purple;

        text-decoration:underline;}

p.MsoListParagraph, li.MsoListParagraph, div.MsoListParagraph

        {mso-style-priority:34;

        margin-top:0cm;

        margin-right:0cm;

        margin-bottom:0cm;

        margin-left:36.0pt;

        margin-bottom:.0001pt;

        font-size:11.0pt;

        font-family:"Calibri","sans-serif";}

p.msonormal0, li.msonormal0, div.msonormal0

        {mso-style-name:msonormal;

        mso-margin-top-alt:auto;

        margin-right:0cm;

        mso-margin-bottom-alt:auto;

        margin-left:0cm;

        font-size:12.0pt;

        font-family:"MS PGothic","sans-serif";}

span.EmailStyle19

        {mso-style-type:personal;

        font-family:"Meiryo UI","sans-serif";

        color:black;

        font-weight:normal;

        font-style:normal;}

span.EmailStyle20

        {mso-style-type:personal;

        font-family:"Calibri","sans-serif";

        color:windowtext;}

span.EmailStyle21

        {mso-style-type:personal-reply;

        font-family:"Meiryo UI","sans-serif";

        color:black;

        font-weight:normal;

        font-style:normal;}

.MsoChpDefault

        {mso-style-type:export-only;

        font-size:10.0pt;}

@page WordSection1

        {size:612.0pt 792.0pt;

        margin:72.0pt 72.0pt 72.0pt 72.0pt;}

div.WordSection1

        {page:WordSection1;}

/* List Definitions */

@list l0

        {mso-list-id:459226449;

        mso-list-template-ids:-126599020;}

ol

        {margin-bottom:0cm;}

ul

        {margin-bottom:0cm;}

--></style><!--[if gte mso 9]><xml>

<o:shapedefaults v:ext="edit" spidmax="1026" />

</xml><![endif]--><!--[if gte mso 9]><xml>

<o:shapelayout v:ext="edit">

<o:idmap v:ext="edit" data="1" />

</o:shapelayout></xml><![endif]-->

</head>

<body lang="EN-PH" link="blue" vlink="purple">

<div class="WordSection1">

<p class="MsoNormal"><span style="font-family:"Meiryo UI","sans-serif";color:black">Hi,<o:p></o:p></span></p>

<p class="MsoNormal"><span style="font-family:"Meiryo UI","sans-serif";color:black"><o:p> </o:p></span></p>

<p class="MsoNormal"><span style="font-family:"Meiryo UI","sans-serif";color:black">After removing support for the i64 type in the *CallingConv.td, sret-demotion is performed and we now have a store<(store 8, align 1)> DAG node being generated. Please refer

 to the attached dag_funcret.pdf DAG visualization.<o:p></o:p></span></p>

<p class="MsoNormal"><span style="font-family:"Meiryo UI","sans-serif";color:black"><o:p> </o:p></span></p>

<p class="MsoNormal"><span style="font-family:"Meiryo UI","sans-serif";color:black">My understanding is that, the second operand(CopyFromReg->Register %1, Register %0 back-up) in the store node is the memory location allocated for the i64 type return. Unfortunately,

 this register is killed in the live variable analysis and makes the Register %0 the second operand. The Register %0 is overwritten during the processing of the function which makes the store node have an incorrect address as the second operand.<o:p></o:p></span></p>

<p class="MsoNormal"><span style="font-family:"Meiryo UI","sans-serif";color:black"><o:p> </o:p></span></p>

<p class="MsoNormal"><span style="font-family:"Meiryo UI","sans-serif";color:black">I have also tried implementing SRET processing similar to x86’s implementation, this saves the Register %0 value into another register but it is stored after the store node

 (starting from the EntryToken). Refer to dag_sret_proc.pdf for the DAG visualization for this.<o:p></o:p></span></p>

<p class="MsoNormal"><span style="font-family:"Meiryo UI","sans-serif";color:black"><o:p> </o:p></span></p>

<p class="MsoNormal"><span style="font-family:"Meiryo UI","sans-serif";color:black">sret-demotion automatically sets er0(16-bit register, copied into Register %0) as a livein function argument containing the return memory location in LowerFormalArguments. er0

 is a sub register of the qr0(i64 register) used in the processing. I think I was not able to inform the backend that if qr0 is modified, er0 is modified as well. Is there a way to do this?<o:p></o:p></span></p>

<p class="MsoNormal"><span style="font-family:"Meiryo UI","sans-serif";color:black"><o:p> </o:p></span></p>

<p class="MsoNormal"><span style="font-family:"Meiryo UI","sans-serif";color:black">What classes/functions should I look into so that Register %1 will not be killed?<o:p></o:p></span></p>

<p class="MsoNormal"><span style="font-family:"Meiryo UI","sans-serif";color:black">I am considering to create a custom select function for ISD::STORE but I am worried that most probably i64 function ret is not the only process that would generate this kind

 of store node.<o:p></o:p></span></p>

<p class="MsoNormal"><span style="font-family:"Meiryo UI","sans-serif";color:black"><o:p> </o:p></span></p>

<p class="MsoNormal"><span style="font-family:"Meiryo UI","sans-serif";color:black">Thank you very much for your help and time!<o:p></o:p></span></p>

<p class="MsoNormal"><span style="font-family:"Meiryo UI","sans-serif";color:black"><o:p> </o:p></span></p>

<p class="MsoNormal"><span style="font-family:"Meiryo UI","sans-serif";color:black">Cheers,<o:p></o:p></span></p>

<p class="MsoNormal"><span style="font-family:"Meiryo UI","sans-serif";color:black">Miguel<o:p></o:p></span></p>

<p class="MsoNormal"><span style="font-family:"Meiryo UI","sans-serif";color:black"><o:p> </o:p></span></p>

<div>

<div style="border:none;border-top:solid #B5C4DF 1.0pt;padding:3.0pt 0cm 0cm 0cm">

<p class="MsoNormal"><b><span lang="EN-US" style="font-size:10.0pt;font-family:"Tahoma","sans-serif"">From:</span></b><span lang="EN-US" style="font-size:10.0pt;font-family:"Tahoma","sans-serif""> Eli Friedman [mailto:efriedma@quicinc.com]

<br>

<b>Sent:</b> February 13, 2020 9:46 AM<br>

<b>To:</b> Miguel Inigo J. Manalac; llvm-dev<br>

<b>Subject:</b> RE: [llvm-dev] Function Return Legalization<o:p></o:p></span></p>

</div>

</div>

<p class="MsoNormal"><o:p> </o:p></p>

<p class="MsoNormal"><span lang="EN-US" style="mso-fareast-language:EN-US">LLVM calls a pointer that’s used to return a value indirectly, like you’re describing, an “sret” argument.  There are sort of two ways to go about generating one. First, you can edit

 the call lowering code in clang (clang/lib/CodeGen/TargetInfo.cpp) so the pointer is represented explicitly in IR.  Second, you can make the target-independent backend code generate an sret argument in SelectionDAG, by making your target’s “CanLowerReturn”

 return false.<o:p></o:p></span></p>

<p class="MsoNormal"><span lang="EN-US" style="mso-fareast-language:EN-US"><o:p> </o:p></span></p>

<p class="MsoNormal"><span lang="EN-US" style="mso-fareast-language:EN-US">Probably making CanLowerReturn return false for i64 makes sense for your target.  If you’re using TableGen’ed calling conventions (*CallingConv.td), the TableGen’ed code will handle

 this automatically; otherwise, you can write it out explicitly in C++. If you want an example of how this works in practice, try something like “echo 'define i128 @foo() { ret i128 3 }' | llc -mtriple=i686”.

<o:p></o:p></span></p>

<p class="MsoNormal"><span lang="EN-US" style="mso-fareast-language:EN-US"><o:p> </o:p></span></p>

<p class="MsoNormal"><span lang="EN-US" style="mso-fareast-language:EN-US">If you need to stick the sret pointer into a special register, you can use CCIfSRet in TableGen.  (Or if you’re not using a TableGen’ed calling convention, you can check IsSRet in C++.)<o:p></o:p></span></p>

<p class="MsoNormal"><span lang="EN-US" style="mso-fareast-language:EN-US"><o:p> </o:p></span></p>

<p class="MsoNormal"><span lang="EN-US" style="mso-fareast-language:EN-US">-Eli<o:p></o:p></span></p>

<p class="MsoNormal"><span lang="EN-US" style="mso-fareast-language:EN-US"><o:p> </o:p></span></p>

<div style="border:none;border-left:solid blue 1.5pt;padding:0cm 0cm 0cm 4.0pt">

<div>

<div style="border:none;border-top:solid #E1E1E1 1.0pt;padding:3.0pt 0cm 0cm 0cm">

<p class="MsoNormal"><b><span lang="EN-US">From:</span></b><span lang="EN-US"> llvm-dev <llvm-dev-bounces@lists.llvm.org>

<b>On Behalf Of </b>Miguel Inigo J. Manalac via llvm-dev<br>

<b>Sent:</b> Wednesday, February 12, 2020 12:32 AM<br>

<b>To:</b> llvm-dev@lists.llvm.org<br>

<b>Subject:</b> [EXT] [llvm-dev] Function Return Legalization<o:p></o:p></span></p>

</div>

</div>

<p class="MsoNormal"><span lang="EN-US"><o:p> </o:p></span></p>

<p class="MsoNormal"><span style="font-family:"Meiryo UI","sans-serif";color:black">Hi All,<o:p></o:p></span></p>

<p class="MsoNormal"><span style="font-family:"Meiryo UI","sans-serif";color:black"><o:p> </o:p></span></p>

<p class="MsoNormal"><span style="font-family:"Meiryo UI","sans-serif";color:black">In the target we are implementing, function return for i64 and f64 types has a different processing.

<o:p></o:p></span></p>

<p class="MsoNormal"><span style="font-family:"Meiryo UI","sans-serif";color:black"><o:p> </o:p></span></p>

<p class="MsoNormal"><span style="font-family:"Meiryo UI","sans-serif";color:black">For types i8 to i32, and f32, the return values are stored in their designated return registers (like how other targets does it).

<o:p></o:p></span></p>

<p class="MsoNormal"><span style="font-family:"Meiryo UI","sans-serif";color:black"><o:p> </o:p></span></p>

<p class="MsoNormal"><span style="font-family:"Meiryo UI","sans-serif";color:black">For i64 and f64 types, in the function call, after pushing the function parameters into the stack, the address of the allocated return memory space is assigned to a 16-bit register

 (In our case, ER0). This is done by assigning the frame pointer register into ER0 and then adding the offset to it. So the output assembly code would somewhat look like the following:<o:p></o:p></span></p>

<p class="MsoNormal"><span style="font-family:"Meiryo UI","sans-serif";color:black"><o:p> </o:p></span></p>

<p class="MsoNormal"><span style="font-family:"Meiryo UI","sans-serif";color:black">Expected Target ASM output for i64 function call with i64 return type:<o:p></o:p></span></p>

<p class="MsoNormal"><span style="font-family:"Meiryo UI","sans-serif";color:black">1    push qr0                    ;; 64-bit parameter pushed into the stack<o:p></o:p></span></p>

<p class="MsoNormal"><span style="font-family:"Meiryo UI","sans-serif";color:black">2    mov er0, fp                ;; Assign the frame pointer into er0<o:p></o:p></span></p>

<p class="MsoNormal"><span style="font-family:"Meiryo UI","sans-serif";color:black">3    add er0, #-32            ;; Add the frame pointer offset to er0<o:p></o:p></span></p>

<p class="MsoNormal"><span style="font-family:"Meiryo UI","sans-serif";color:black">4    bl _fn                        ;; Function call fn<o:p></o:p></span></p>

<p class="MsoNormal"><span style="font-family:"Meiryo UI","sans-serif";color:black">5    add sp, #8                 ;; SP adjustment<o:p></o:p></span></p>

<p class="MsoNormal"><span style="font-family:"Meiryo UI","sans-serif";color:black"><o:p> </o:p></span></p>

<p class="MsoNormal"><span style="font-family:"Meiryo UI","sans-serif";color:black">In function fn:<o:p></o:p></span></p>

<p class="MsoNormal"><span style="font-family:"Meiryo UI","sans-serif";color:black">1 ;; function prologue processing here...<o:p></o:p></span></p>

<p class="MsoNormal"><span style="font-family:"Meiryo UI","sans-serif";color:black">2 mov er8, er0                 ;; frame pointer location of retval is transferred to er8 (will be used at line 4)<o:p></o:p></span></p>

<p class="MsoNormal"><span style="font-family:"Meiryo UI","sans-serif";color:black">3 ;; function processing here... Assuming that the i64 value to be returned is stored at qr0...<o:p></o:p></span></p>

<p class="MsoNormal"><span style="font-family:"Meiryo UI","sans-serif";color:black">4 lea [er8]                       ;;  i64 return processing starts here. Load effective address from [er8]<o:p></o:p></span></p>

<p class="MsoNormal"><span style="font-family:"Meiryo UI","sans-serif";color:black">5 st qr0, [ea]                   ;; Store the i64 value into the effective address memory location.

<o:p></o:p></span></p>

<p class="MsoNormal"><span style="font-family:"Meiryo UI","sans-serif";color:black">6 ;; function epilogue processing here...<o:p></o:p></span></p>

<p class="MsoNormal"><span style="font-family:"Meiryo UI","sans-serif";color:black"><o:p> </o:p></span></p>

<p class="MsoNormal"><span style="font-family:"Meiryo UI","sans-serif";color:black">From my investigations so far, i64 types should call a custom function in the calling convention for processing this return type. Also, that LowerCall should be modified to

 have the assigning of frame pointer into er0 and that LowerReturn should be modified in order to store qr0 into the effective address. Please correct me if I<span lang="JA">’</span>m wrong.<o:p></o:p></span></p>

<p class="MsoNormal"><span style="font-family:"Meiryo UI","sans-serif";color:black"><o:p> </o:p></span></p>

<p class="MsoNormal"><span style="font-family:"Meiryo UI","sans-serif";color:black">My questions are:<o:p></o:p></span></p>

<ol style="margin-top:0cm" start="1" type="1">

<li class="MsoNormal" style="color:black;mso-list:l0 level1 lfo1"><span style="font-family:"Meiryo UI","sans-serif"">Are there any target examples that also has this kind of behavior? If there are please let me know so I could study on it.<o:p></o:p></span></li><li class="MsoNormal" style="color:black;mso-list:l0 level1 lfo1"><span style="font-family:"Meiryo UI","sans-serif"">Where does the assigning of frame pointer+offset(frame index?) fall? I am currently suspecting that it should be in the LowerCall function.

<o:p></o:p></span></li><li class="MsoNormal" style="color:black;mso-list:l0 level1 lfo1"><span style="font-family:"Meiryo UI","sans-serif"">If my assumption that question #2 falls in LowerCall function is correct, how do I retrieve the created frame index from the LowerCall function

 in the LowerReturn function?<o:p></o:p></span></li><li class="MsoNormal" style="color:black;mso-list:l0 level1 lfo1"><span style="font-family:"Meiryo UI","sans-serif"">I also suspect that the affected functions are not limited to LowerCall and LowerReturn, if you have suggestions on which ISelLowering Class

 functions I should investigate on, please let me know.<o:p></o:p></span></li></ol>

<p class="MsoNormal"><span style="font-family:"Meiryo UI","sans-serif";color:black"><o:p> </o:p></span></p>

<p class="MsoNormal"><span style="font-family:"Meiryo UI","sans-serif";color:black">Thank you very much in advance for your help!<o:p></o:p></span></p>

<p class="MsoNormal"><span style="font-family:"Meiryo UI","sans-serif";color:black"><o:p> </o:p></span></p>

<p class="MsoNormal"><span style="font-family:"Meiryo UI","sans-serif";color:black">Sincerely,<o:p></o:p></span></p>

<p class="MsoNormal"><span style="font-family:"Meiryo UI","sans-serif";color:black">Miguel Inigo J. Manalac<o:p></o:p></span></p>

<p class="MsoNormal"><o:p> </o:p></p>

<p class="MsoNormal"><span style="font-size:12.0pt;font-family:"MS PGothic","sans-serif"">JAPANESE:

</span><span lang="JA" style="font-size:12.0pt;font-family:"MS PGothic","sans-serif"">このメ・踉札襪蓮���宛先に書かれている方のみに送信することを意図しております。誤ってそれ以外の方に送信された��豺腓蓮���申し訳ございませんが・髟阡餐��踉市までお知らせいただき・髟阡殺踉歯信されたメ・踉札襪鮑鐔釮靴討い燭世④泙垢茲�逅擦�蠅いい燭靴泙后�泙拭���コンピュ・踉札拭�逅札�逅札�襯垢虜��銈砲茲辰謄瓠��ルの欠落・瘢雹不整合・瘢雹遅滞等が発生し・髟阡参燭蕕里管塋悗鬚��韻垢襪海箸�犬犬討睚畝踉子はその責任を負いません。</span><span style="font-size:12.0pt;font-family:"MS PGothic","sans-serif"">

 ENGLISH: This e-mail is intended for the person(s) to which it is addressed. If you have received it by mistake, please notify the sender and delete the received email. In addition, our company shall not assume any responsibility even if it causes any inconvenience,

 such as loss of mail, inconsistencies, delays, etc., due to the inclusion of computer viruses.

<o:p></o:p></span></p>

</div>

</div>

JAPANESE: このメ・踉札襪蓮���宛先に書かれている方のみに送信することを意図しております。誤ってそれ以外の方に送信された��豺腓蓮���申し訳ございませんが・髟阡餐��踉市までお知らせいただき・髟阡殺踉歯信されたメ・踉札襪鮑鐔釮靴討い燭世④泙垢茲�逅擦�蠅いい燭靴泙后�泙拭���コンピュ・踉札拭�逅札�逅札�襯垢虜��銈砲茲辰謄瓠��ルの欠落・瘢雹不整合・瘢雹遅滞等が発生し・髟阡参燭蕕里管塋悗鬚��韻垢襪海箸�犬犬討睚畝踉子はその責任を負いません。 ENGLISH: This e-mail is intended for the person(s) to which it

 is addressed. If you have received it by mistake, please notify the sender and delete the received email. In addition, our company shall not assume any responsibility even if it causes any inconvenience, such as loss of mail, inconsistencies, delays, etc.,

 due to the inclusion of computer viruses.

</body>

</html>