[PATCH][X86_64/MC] Towards usable Win64 exception handling

Kai Nacke kai.nacke at redstar.de
Sun Jul 7 21:52:33 PDT 2013


Hi Chandler,

sorry for that. I was under the impression that I followed the review 
process but now I know better. Won't happen again.

Again, sorry for that.

Kai

On 08.07.2013 03:05, Chandler Carruth wrote:
> 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.
>
> 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.
>
> 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.
>
> (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.)
>
>
> On Sat, Jul 6, 2013 at 10:25 AM, Kai Nacke <kai.nacke at redstar.de
> <mailto:kai.nacke at redstar.de>> wrote:
>
>     Thanks for the review.
>
>     Commited in r185758..r185761
>
>     Kai
>
>
>     On 03.07.2013 22:45, Charles Davis wrote:
>
>
>         On Jul 3, 2013, at 10:28 AM, Kai Nacke wrote:
>
>             Hi!
>
>             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.
>
>             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.
>
>         All four LGTM.
>
>         Chip
>
>
>         _________________________________________________
>         llvm-commits mailing list
>         llvm-commits at cs.uiuc.edu <mailto:llvm-commits at cs.uiuc.edu>
>         http://lists.cs.uiuc.edu/__mailman/listinfo/llvm-commits
>         <http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits>
>
>
>     _________________________________________________
>     llvm-commits mailing list
>     llvm-commits at cs.uiuc.edu <mailto:llvm-commits at cs.uiuc.edu>
>     http://lists.cs.uiuc.edu/__mailman/listinfo/llvm-commits
>     <http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits>
>
>




More information about the llvm-commits mailing list