<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=us-ascii">
<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:Consolas;
panose-1:2 11 6 9 2 2 4 3 2 4;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
{margin:0in;
margin-bottom:.0001pt;
font-size:12.0pt;
font-family:"Times New Roman","serif";
color:black;}
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;}
code
{mso-style-priority:99;
font-family:"Courier New";}
pre
{mso-style-priority:99;
mso-style-link:"HTML Preformatted Char";
margin:0in;
margin-bottom:.0001pt;
font-size:10.0pt;
font-family:"Courier New";
color:black;}
span.HTMLPreformattedChar
{mso-style-name:"HTML Preformatted Char";
mso-style-priority:99;
mso-style-link:"HTML Preformatted";
font-family:"Consolas","serif";
color:black;}
span.EmailStyle20
{mso-style-type:personal-reply;
font-family:"Calibri","sans-serif";
color:#1F497D;}
.MsoChpDefault
{mso-style-type:export-only;
font-size:10.0pt;}
@page WordSection1
{size:8.5in 11.0in;
margin:1.0in 1.0in 1.0in 1.0in;}
div.WordSection1
{page:WordSection1;}
--></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 bgcolor="white" lang="EN-US" link="blue" vlink="purple">
<div class="WordSection1">
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D">Hi Alp,<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D"><o:p> </o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D">The ‘.ir’ extension, as I’m using it, is exactly equivalent to the ‘.ll’ extension. The reason I used it was to have a way to specify additional files that
would be used to provide secondary module definitions in lit tests without the testing framework trying to use them as test definitions themselves. If there is a better way to do this, I’d be happy to change it.<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D"><o:p> </o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D">As for the related ‘-extra-modules’ flag added to lli, I’ll update it in accordance with your suggestion.<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D"><o:p> </o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D">Thanks,<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D">Andy<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D"><o:p> </o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D"><o:p> </o:p></span></p>
<div>
<div style="border:none;border-top:solid #B5C4DF 1.0pt;padding:3.0pt 0in 0in 0in">
<p class="MsoNormal"><b><span style="font-size:10.0pt;font-family:"Tahoma","sans-serif";color:windowtext">From:</span></b><span style="font-size:10.0pt;font-family:"Tahoma","sans-serif";color:windowtext"> Alp Toker [mailto:alp@nuanti.com]
<br>
<b>Sent:</b> Monday, October 28, 2013 2:34 PM<br>
<b>To:</b> Kaylor, Andrew<br>
<b>Cc:</b> cfe commits<br>
<b>Subject:</b> lli and the .ir file extension<o:p></o:p></span></p>
</div>
</div>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">Hello Andrew,<br>
<br>
In r191938, r192021 and r192504 you introduced a new file extension ".ir" to the LLVM project.<br>
<br>
Could you document this extension and how it differs from the existing ".ll" format, or otherwise update the tools to use one of the two existing well-known extensions ".ll" or ".bc"?<br>
<br>
Additionally, the "-extra-modules" flag added to lli in r191938 takes a comma-separated list of filenames(!)<br>
<br>
It'd be better to handle this the usual way with something like "-module fileA.ext -module fileB.ext" following the convention of other commands. This should just be a matter of removing cl::CommaSeparated from the cl::opt in lli.cpp.<br>
<br>
Background: I've been adding a validator to lit and these two have needed hacky special-casing so far.<br>
<br>
Related files:<br>
<br>
<code><span style="font-size:10.0pt"> test/ExecutionEngine/MCJIT/cross-module-b.ir</span></code><span style="font-size:10.0pt;font-family:"Courier New""><br>
<code> test/ExecutionEngine/MCJIT/multi-module-b.ir</code><br>
<code> test/ExecutionEngine/MCJIT/multi-module-c.ir</code><br>
<code> test/ExecutionEngine/MCJIT/multi-module-eh-b.ir</code><br>
<code> test/ExecutionEngine/MCJIT/remote/cross-module-b.ir</code><br>
<code> test/ExecutionEngine/MCJIT/remote/multi-module-b.ir</code><br>
<code> test/ExecutionEngine/MCJIT/remote/multi-module-c.ir</code></span><br>
<br>
<br>
Alp.<br>
<br>
<br>
<o:p></o:p></p>
<pre>-- <o:p></o:p></pre>
<pre><a href="http://www.nuanti.com">http://www.nuanti.com</a><o:p></o:p></pre>
<pre>the browser experts<o:p></o:p></pre>
</div>
</body>
</html>