<div dir="ltr">Kai, I hate to say this but these patches should not be committed based on Charles and Nico's review alone. While their review helps, you will need one of the long-time committers to the codegen area to give final approval on the design and structure of these patches.<div>
<br></div><div>Please back this patch series out and ping the list asking for one of the existing maintainers to look at the patches. Notably, the last long-time maintainer to look at this patch series (Anton) had questions and didn't express that this LGTM.</div>
<div><br></div><div>I'm sorry the process is extended and it is hard to get good reviews, but letting patches in without thorough review doesn't solve the problem, it just lowers the quality standards for the project.</div>
<div><br></div><div>(For the record, I'm not saying there *is* a problem with these patches, I'm saying that we need to follow a reasonable practice here and get good review on the design before committing.)</div>
</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Sat, Jul 6, 2013 at 10:25 AM, Kai Nacke <span dir="ltr"><<a href="mailto:kai.nacke@redstar.de" target="_blank">kai.nacke@redstar.de</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Thanks for the review.<br>
<br>
Commited in r185758..r185761<span class="HOEnZb"><font color="#888888"><br>
<br>
Kai</font></span><div class="HOEnZb"><div class="h5"><br>
<br>
On 03.07.2013 22:45, Charles Davis wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
On Jul 3, 2013, at 10:28 AM, Kai Nacke wrote:<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi!<br>
<br>
Please, I still need a review for the set of attached patches. They are really needed for working Win64 SEH support. Implementation of Dwarf EH handling on top of Win64 SEH is then a follow-up patch.<br>
<br>
The first patch fixes the now failing seh.s test. The second patch changes the relocation type to IMAGE_REL_AMD64_ADDR32NB as required by the MS COFF spec. The third patch addresses an alignment problem inside the unwind info data structure (this one is new). The fourth patch outputs the personality function and the language data into the right places.<br>

</blockquote>
All four LGTM.<br>
<br>
Chip<br>
<br>
<br>
______________________________<u></u>_________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/<u></u>mailman/listinfo/llvm-commits</a><br>
<br>
</blockquote>
<br>
______________________________<u></u>_________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/<u></u>mailman/listinfo/llvm-commits</a><br>
</div></div></blockquote></div><br></div>