<div class="gmail_quote">On 2 May 2010 00:59, Nathan Jeffords <span dir="ltr"><<a href="mailto:blunted2night@gmail.com">blunted2night@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">

Hi All,<div><br></div><div>I have created a minimally functional Win32 COFF Exporter using the new MC framework. I made some minor changes to other libraries to allow me to plug it in without building it as part of the LLVM project. I wanted to share it but wasn't sure how to go about doing so, so I have attached the code to this message.</div>


<div><br></div><div>Any feedback on would be appreciated.</div></blockquote><div><br></div><div>Hi Nathan,</div><div><br></div><div>A couple of points :-</div><div><br></div><div>1) Your code seems incomplete, there is nothing envoking 'createWin32CoffStreamer()' in you patch. </div>

<div>2) You need to look at LLVM coding standards, i.e. maximum line length of 80 characters and tabs of 2. You need to look at the following document and existing code. Usage of spaces is quite strict.</div><div><br></div>

<div>   <a href="http://llvm.org/docs/CodingStandards.html">http://llvm.org/docs/CodingStandards.html</a></div><div><br></div><div>I too have been working on a COFFWriter but with little success. Got a basic COFFWriter working but have had problems with the MC framework in regards to getting the right information to generate code properly.</div>

<div><br></div><div>It will be good if we can get something working first, so send me a proper change set, read this :-</div><div><br></div><div>    <a href="http://llvm.org/docs/DeveloperPolicy.html">http://llvm.org/docs/DeveloperPolicy.html</a></div>

<div>   </div><div>I generally use :-</div><div><br></div><div><font class="Apple-style-span" face="'courier new', monospace">    utils/mkpatch patchname</font></div><div><br></div><div>from the root svn directory with out the '.patch' to make patches. You need to include any files from 'svn status' with question marks, by doing an 'svn add filename'.</div>

<div><br></div><div>Thanks for submitting a patch, most patches are welcomed, but they may need to be honed a bit.</div><div><br></div><div>Aaron</div><div><br></div></div>