<html>
<head>
<meta http-equiv="content-type" content="text/html; charset=ISO-8859-1">
</head>
<body bgcolor="#FFFFFF" text="#000000">
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> test/ExecutionEngine/MCJIT/cross-module-b.ir</code><code><br>
</code><code> test/ExecutionEngine/MCJIT/multi-module-b.ir</code><code><br>
</code><code> test/ExecutionEngine/MCJIT/multi-module-c.ir</code><code><br>
</code><code> test/ExecutionEngine/MCJIT/multi-module-eh-b.ir</code><code><br>
</code><code> test/ExecutionEngine/MCJIT/remote/cross-module-b.ir</code><code><br>
</code><code> test/ExecutionEngine/MCJIT/remote/multi-module-b.ir</code><code><br>
</code><code> test/ExecutionEngine/MCJIT/remote/multi-module-c.ir</code><br>
<br>
<br>
Alp.<br>
<br>
<pre class="moz-signature" cols="72">--
<a class="moz-txt-link-freetext" href="http://www.nuanti.com">http://www.nuanti.com</a>
the browser experts
</pre>
</body>
</html>