<div dir="ltr">I just merged the pull request.<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, May 11, 2017 at 5:27 PM,  <span dir="ltr"><<a href="mailto:hemant@mnkcg.com" target="_blank">hemant@mnkcg.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div link="blue" vlink="purple" lang="EN-US"><div class="m_-574864315423878250WordSection1"><p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif">Great, thanks much!<u></u><u></u></span></p><p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif"><u></u> <u></u></span></p><p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif">Hemant<u></u><u></u></span></p><p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif"><u></u> <u></u></span></p><div><div style="border:none;border-top:solid #e1e1e1 1.0pt;padding:3.0pt 0in 0in 0in"><p class="MsoNormal"><b><span style="font-size:11.0pt;font-family:"Calibri",sans-serif">From:</span></b><span style="font-size:11.0pt;font-family:"Calibri",sans-serif"> Mihai Budiu [mailto:<a href="mailto:mbudiu@vmware.com" target="_blank">mbudiu@vmware.com</a>] <br><b>Sent:</b> Thursday, May 11, 2017 8:24 PM<br><b>To:</b> <a href="mailto:hemant@mnkcg.com" target="_blank">hemant@mnkcg.com</a>; 'Antonin Bas' <<a href="mailto:antonin@barefootnetworks.com" target="_blank">antonin@barefootnetworks.com</a>><br><b>Cc:</b> 'p4-dev' <<a href="mailto:p4-dev@lists.p4.org" target="_blank">p4-dev@lists.p4.org</a>></span></p><div><div class="h5"><br><b>Subject:</b> RE: [P4-dev] P4-16 HLIR with struct support?<u></u><u></u></div></div><p></p></div></div><div><div class="h5"><p class="MsoNormal"><u></u> <u></u></p><p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif;color:#1f497d">The pull request is in the queue, has been approved and is awaiting someone to click the “merge” button.<u></u><u></u></span></p><p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif;color:#1f497d">I will rebase it on master to make it easier; after each rebase the tests have to run again…<u></u><u></u></span></p><p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif;color:#1f497d"><u></u> <u></u></span></p><p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif;color:#1f497d">Mihai<u></u><u></u></span></p><p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif;color:#1f497d"><u></u> <u></u></span></p><div><div style="border:none;border-top:solid #e1e1e1 1.0pt;padding:3.0pt 0in 0in 0in"><p class="MsoNormal"><b><span style="font-size:11.0pt;font-family:"Calibri",sans-serif">From:</span></b><span style="font-size:11.0pt;font-family:"Calibri",sans-serif"> <a href="mailto:hemant@mnkcg.com" target="_blank">hemant@mnkcg.com</a> [<a href="mailto:hemant@mnkcg.com" target="_blank">mailto:hemant@mnkcg.com</a>] <br><b>Sent:</b> Thursday, May 11, 2017 5:21 PM<br><b>To:</b> 'Antonin Bas' <<a href="mailto:antonin@barefootnetworks.com" target="_blank">antonin@barefootnetworks.com</a>><wbr>; Mihai Budiu <<a href="mailto:mbudiu@vmware.com" target="_blank">mbudiu@vmware.com</a>><br><b>Cc:</b> 'p4-dev' <<a href="mailto:p4-dev@lists.p4.org" target="_blank">p4-dev@lists.p4.org</a>><br><b>Subject:</b> RE: [P4-dev] P4-16 HLIR with struct support?<u></u><u></u></span></p></div></div><p class="MsoNormal"><u></u> <u></u></p><p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif">The issue I filed was #583 and thanks to Mihai who expedited the fix and committed the code to a p4c-clone branch.   So how soon can I expect this fix to be in the mainline p4lang/p4c?   I found the cloned branch below but this not look to be a release vehicle I can use to get this fix, right?<u></u><u></u></span></p><p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif"><u></u> <u></u></span></p><p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif"><a href="https://github.com/mbudiu-vmw/p4c-clone/tree/issue583" target="_blank">https://github.com/mbudiu-vmw/<wbr>p4c-clone/tree/issue583</a><u></u><u></u></span></p><p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif"><u></u> <u></u></span></p><p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif">thanks,<u></u><u></u></span></p><p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif"><u></u> <u></u></span></p><p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif">Hemant<u></u><u></u></span></p><p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif"><u></u> <u></u></span></p><div><div style="border:none;border-top:solid #e1e1e1 1.0pt;padding:3.0pt 0in 0in 0in"><p class="MsoNormal"><b><span style="font-size:11.0pt;font-family:"Calibri",sans-serif">From:</span></b><span style="font-size:11.0pt;font-family:"Calibri",sans-serif"> P4-dev [<a href="mailto:p4-dev-bounces@lists.p4.org" target="_blank">mailto:p4-dev-bounces@lists.<wbr>p4.org</a>] <b>On Behalf Of </b><a href="mailto:hemant@mnkcg.com" target="_blank">hemant@mnkcg.com</a><br><b>Sent:</b> Monday, May 08, 2017 7:06 PM<br><b>To:</b> 'Antonin Bas' <<a href="mailto:antonin@barefootnetworks.com" target="_blank">antonin@barefootnetworks.com</a>><br><b>Cc:</b> 'p4-dev' <<a href="mailto:p4-dev@lists.p4.org" target="_blank">p4-dev@lists.p4.org</a>><br><b>Subject:</b> Re: [P4-dev] P4-16 HLIR with struct support?<u></u><u></u></span></p></div></div><p class="MsoNormal"><u></u> <u></u></p><p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif">Ok, thanks.  I will open an issue – thanks.<u></u><u></u></span></p><p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif"><u></u> <u></u></span></p><p class="MsoNormal"><b><span style="font-size:11.0pt;font-family:"Calibri",sans-serif">From:</span></b><span style="font-size:11.0pt;font-family:"Calibri",sans-serif"> Antonin Bas [</span><a href="mailto:antonin@barefootnetworks.com" target="_blank"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif">mailto:antonin@<wbr>barefootnetworks.com</span></a><span style="font-size:11.0pt;font-family:"Calibri",sans-serif">] <br><b>Sent:</b> Monday, May 08, 2017 1:05 PM<br><b>To:</b> </span><a href="mailto:hemant@mnkcg.com" target="_blank"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif">hemant@mnkcg.com</span></a><span style="font-size:11.0pt;font-family:"Calibri",sans-serif"><br><b>Cc:</b> Mihai Budiu <</span><a href="mailto:mbudiu@vmware.com" target="_blank"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif">mbudiu@vmware.com</span></a><span style="font-size:11.0pt;font-family:"Calibri",sans-serif">>; p4-dev <</span><a href="mailto:p4-dev@lists.p4.org" target="_blank"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif">p4-dev@lists.p4.org</span></a><span style="font-size:11.0pt;font-family:"Calibri",sans-serif">><br><b>Subject:</b> Re: [P4-dev] P4-16 HLIR with struct support?<u></u><u></u></span></p><p class="MsoNormal"><u></u> <u></u></p><div><p class="MsoNormal">Compilation fails for me as well, but with a different error message:<u></u><u></u></p><div><p class="MsoNormal"><u></u> <u></u></p></div><div><div><p class="MsoNormal">includes/headers.p4(18): error: header ipv4_t: Duplicates declaration header ipv4_t<u></u><u></u></p></div><div><p class="MsoNormal">header_type ipv4_t {<u></u><u></u></p></div><div><p class="MsoNormal">^<u></u><u></u></p></div><div><p class="MsoNormal">includes/headers.p4(18)<u></u><u></u></p></div><div><p class="MsoNormal">header_type ipv4_t {<u></u><u></u></p></div><div><p class="MsoNormal">^<u></u><u></u></p></div><div><p class="MsoNormal">error: 1 errors encountered, aborting compilation<u></u><u></u></p></div></div><div><p class="MsoNormal"><u></u> <u></u></p></div><div><p class="MsoNormal">You may want to open an issue in p4lang/p4c<u></u><u></u></p></div></div><div><p class="MsoNormal"><u></u> <u></u></p><div><p class="MsoNormal">On Sun, May 7, 2017 at 2:53 PM, <<a href="mailto:hemant@mnkcg.com" target="_blank">hemant@mnkcg.com</a>> wrote:<u></u><u></u></p><blockquote style="border:none;border-left:solid #cccccc 1.0pt;padding:0in 0in 0in 6.0pt;margin-left:4.8pt;margin-top:5.0pt;margin-right:0in;margin-bottom:5.0pt"><div><div><p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif">In the p4-hlir/tests directory, there is a stateful.p4 file (</span><a href="https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_p4lang_p4-2Dhlir_blob_master_tests_stateful.p4&d=DwMFaQ&c=uilaK90D4TOVoH58JNXRgQ&r=tGW6TKXajnoXSyy1S1P4DHGPe8sj54GGvw-b21n7aWg&m=QQR_rqeCkQ57g5qbv0S5qybtBwnNYlmIYGG8bzVU3hU&s=wdDzic1TiRjb0oaV97B63m3EIMShHpklidoZgzGR1Uk&e=" target="_blank"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif">https://github.com/p4lang/p4-<wbr>hlir/blob/master/tests/<wbr>stateful.p4</span></a><span style="font-size:11.0pt;font-family:"Calibri",sans-serif">).</span><u></u><u></u></p><p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif"> </span><u></u><u></u></p><p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif">I used the following p4c compilation.  The compiler fails to compile this P4-14 file.  Did I do anything wrong using the p4c compiler to compile P4-14 code?  </span><u></u><u></u></p><p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Courier New"">~/p4-hlir/tests$ p4c -S -x p4-14 stateful.p4 </span><u></u><u></u></p><p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Courier New""> </span><u></u><u></u></p><p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Courier New""> </span><u></u><u></u></p><p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Courier New"">includes/parser.p4(46): error: : No header type ipv4_t</span><u></u><u></u></p><p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Courier New"">header ipv4_t ipv4;</span><u></u><u></u></p><p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Courier New"">^^^^^^^^^^^^^^^^^^^</span><u></u><u></u></p><p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Courier New"">includes/parser.p4(46)</span><u></u><u></u></p><p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Courier New"">header ipv4_t ipv4;</span><u></u><u></u></p><p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Courier New"">       ^^^^^^</span><u></u><u></u></p><p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Courier New"">error: 1 errors encountered, aborting compilation</span><u></u><u></u></p><p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Courier New""> </span><u></u><u></u></p><p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Courier New"">~/p4-hlir/tests$</span><u></u><u></u></p><p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif"> </span><u></u><u></u></p><p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif">Best,</span><u></u><u></u></p><p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif"> </span><u></u><u></u></p><p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif">Hemant</span><u></u><u></u></p><p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif"> </span><u></u><u></u></p><p class="MsoNormal"><b><span style="font-size:11.0pt;font-family:"Calibri",sans-serif">From:</span></b><span style="font-size:11.0pt;font-family:"Calibri",sans-serif"> Antonin Bas [mailto:</span><a href="mailto:antonin@barefootnetworks.com" target="_blank"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif">antonin@<wbr>barefootnetworks.com</span></a><span style="font-size:11.0pt;font-family:"Calibri",sans-serif">] <br><b>Sent:</b> Monday, May 01, 2017 11:19 PM<br><b>To:</b> </span><a href="mailto:hemant@mnkcg.com" target="_blank"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif">hemant@mnkcg.com</span></a><span style="font-size:11.0pt;font-family:"Calibri",sans-serif"><br><b>Cc:</b> p4-dev <</span><a href="mailto:p4-dev@lists.p4.org" target="_blank"><span style="font-size:11.0pt;font-family:"Calibri",sans-serif">p4-dev@lists.p4.org</span></a><span style="font-size:11.0pt;font-family:"Calibri",sans-serif">><br><b>Subject:</b> Re: [P4-dev] P4-16 HLIR with struct support?</span><u></u><u></u></p><p class="MsoNormal"> <u></u><u></u></p><div><p class="MsoNormal">Hi Hemant,<u></u><u></u></p><div><div><div><p class="MsoNormal"> <u></u><u></u></p></div><div><p class="MsoNormal">There is no plan to update p4-hlir to support P4_16.<u></u><u></u></p></div><div><p class="MsoNormal">Instead we encourage people to move to the new p4lang/p4c compiler, written in C++, which supports both P4_14 and P4_16: <a href="https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_p4lang_p4c&d=DwMFaQ&c=uilaK90D4TOVoH58JNXRgQ&r=tGW6TKXajnoXSyy1S1P4DHGPe8sj54GGvw-b21n7aWg&m=QQR_rqeCkQ57g5qbv0S5qybtBwnNYlmIYGG8bzVU3hU&s=QmieA6_5Bj2-YP017bNz-I3HrtiH_LeNlgb4EboNZ0Y&e=" target="_blank">https://github.com/<wbr>p4lang/p4c</a><u></u><u></u></p></div><div><p class="MsoNormal">We will keep maintaining p4-hlir for the time being, but no new features will be added.<u></u><u></u></p></div><div><p class="MsoNormal"> <u></u><u></u></p></div><div><p class="MsoNormal">Best,<u></u><u></u></p></div><div><p class="MsoNormal"> <u></u><u></u></p></div><div><p class="MsoNormal">Antonin<u></u><u></u></p></div></div></div></div><div><div><div><p class="MsoNormal"> <u></u><u></u></p><div><p class="MsoNormal">On Mon, May 1, 2017 at 7:16 PM, <<a href="mailto:hemant@mnkcg.com" target="_blank">hemant@mnkcg.com</a>> wrote:<u></u><u></u></p><blockquote style="border:none;border-left:solid #cccccc 1.0pt;padding:0in 0in 0in 6.0pt;margin-left:4.8pt;margin-top:5.0pt;margin-right:0in;margin-bottom:5.0pt"><div><div><p class="MsoNormal">I am interested is using a P4-16 struct in a test program with p4-hlir.  I don’t see the HLIR updated yet with struct from P4-16.   <u></u><u></u></p><p class="MsoNormal"> <u></u><u></u></p><p class="MsoNormal">Hemant<u></u><u></u></p></div></div><p class="MsoNormal"><br>______________________________<wbr>_________________<br>P4-dev mailing list<br><a href="mailto:P4-dev@lists.p4.org" target="_blank">P4-dev@lists.p4.org</a><br><a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__lists.p4.org_mailman_listinfo_p4-2Ddev-5Flists.p4.org&d=DwMFaQ&c=uilaK90D4TOVoH58JNXRgQ&r=tGW6TKXajnoXSyy1S1P4DHGPe8sj54GGvw-b21n7aWg&m=QQR_rqeCkQ57g5qbv0S5qybtBwnNYlmIYGG8bzVU3hU&s=AsdKpIAvMWIT8dxuu3-4-tBPv96_EWZKx9xsaS2FU4k&e=" target="_blank">http://lists.p4.org/mailman/<wbr>listinfo/p4-dev_lists.p4.org</a><u></u><u></u></p></blockquote></div><p class="MsoNormal"><br><br clear="all"><u></u><u></u></p><div><p class="MsoNormal"> <u></u><u></u></p></div><p class="MsoNormal">-- <u></u><u></u></p><div><div><p class="MsoNormal">Antonin<u></u><u></u></p></div></div></div></div></div></div></div></blockquote></div><p class="MsoNormal"><br><br clear="all"><u></u><u></u></p><div><p class="MsoNormal"><u></u> <u></u></p></div><p class="MsoNormal">-- <u></u><u></u></p><div><div><p class="MsoNormal">Antonin<u></u><u></u></p></div></div></div></div></div></div></div></blockquote></div><br><br clear="all"><br>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr">Antonin<br></div></div>
</div>