Hi Chandler, There were five SEH releated patches posted in two threads in the last days. Two different patches in Martell e-mail starting this thread: the win64 seh (llvm) and the register names Three more related SEH patches in another thread: one for win64 seh clang, one for MinGW toolchain and another for unreachable prologue. To clarify and allow proper reviews for the different patches I opened reviews for four of them (the fifth got LGTM in the discussion but it does have the ownership issue you wrote) D3417 Emit a trap instruction for IR 'unreachable <http://reviews.llvm.org/D3417> D3418 SEH exceptions on Win64 (LLVM) <http://reviews.llvm.org/D3418> D3419 SEH exceptions on Win64 (clang part) <http://reviews.llvm.org/D3419> D3420 MinGW toolchain <http://reviews.llvm.org/D3420> Per your suggestion the link to the original discussion in D3418 was added. Code authors are: unreachable is by Vadim Chugunov. win64 seh llvm is by Kai Nacke + Martell re-posting. win64 seh clang is by Martell. mingw toolchain is by Martell. register names is by Ray Donnelly + Martell posting it. seh test for "register names" ( test is r206566) by myself. All but the "register names" patch were posted by their author on the llvm lists. Yaron -------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20140418/67f61883/attachment.html>
Hi chandler, Sorry for the confusion. I've never submitted patches before. I'll cc Ray to get him to confirm that the patch can be merged. His header should be on the patch also so that's how you know it is his. Yes Yaron the names above seem and the detail seem correct to me except that the MinGW driver derived from Rubens submission to LLVM reviews some months ago. I said that at the start of the thread. Basically the guys including Ray were looking to have an x64 mingw64 working with clang. So I fixed Rubens patch and kai's patch for head I'm not concerned with having my name on any attribution I just want them all merged as this combination of patches fixes mingw64 x64 with clang. Kind Regards Martell On 18 Apr 2014 11:11, "Yaron Keren" <yaron.keren at gmail.com> wrote:> Hi Chandler, > > There were five SEH releated patches posted in two threads in the last > days. > > Two different patches in Martell e-mail starting this thread: the win64 > seh (llvm) and the register names > > Three more related SEH patches in another thread: one for win64 seh clang, > one for MinGW toolchain and another for unreachable prologue. > > To clarify and allow proper reviews for the different patches I opened > reviews for four of them (the fifth got LGTM in the discussion but it does > have the ownership issue you wrote) > > D3417 > Emit a trap instruction for IR 'unreachable<http://reviews.llvm.org/D3417> > > D3418 > SEH exceptions on Win64 (LLVM) <http://reviews.llvm.org/D3418> > > D3419 > SEH exceptions on Win64 (clang part) <http://reviews.llvm.org/D3419> > > D3420 > MinGW toolchain <http://reviews.llvm.org/D3420> > > > Per your suggestion the link to the original discussion in D3418 was added. > > Code authors are: > unreachable is by Vadim Chugunov. > win64 seh llvm is by Kai Nacke + Martell re-posting. > win64 seh clang is by Martell. > mingw toolchain is by Martell. > register names is by Ray Donnelly + Martell posting it. > seh test for "register names" ( test is r206566) by myself. > > All but the "register names" patch were posted by their author on the > llvm lists. > > Yaron > > > >-------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20140418/e9984133/attachment.html>
In summary we have no less than six patches required to support Win64 SEH MinGW. The first five could be committed after review and LGTM but the last one also requires Ray Donnelly approval. Please comment in the Phabricator so the comments would be kept in context. 'unreachable' trap http://reviews.llvm.org/D3417 Win64 SEH (LLVM) http://reviews.llvm.org/D3418 Win64 SEH (clang) http://reviews.llvm.org/D3419 MinGW toolchain http://reviews.llvm.org/D3420 TLS (clang) http://reviews.llvm.org/D3421 Register names instead of numbers http://reviews.llvm.org/D3422 2014-04-18 14:42 GMT+03:00 Martell Malone <martellmalone at gmail.com>:> Hi chandler, > > Sorry for the confusion. > I've never submitted patches before. > I'll cc Ray to get him to confirm that the patch can be merged. His header > should be on the patch also so that's how you know it is his. > > Yes Yaron the names above seem and the detail seem correct to me except > that the MinGW driver derived from Rubens submission to LLVM reviews some > months ago. > I said that at the start of the thread. > > Basically the guys including Ray were looking to have an x64 mingw64 > working with clang. So I fixed Rubens patch and kai's patch for head > > I'm not concerned with having my name on any attribution I just want them > all merged as this combination of patches fixes mingw64 x64 with clang. > > Kind Regards > Martell > On 18 Apr 2014 11:11, "Yaron Keren" <yaron.keren at gmail.com> wrote: > >> Hi Chandler, >> >> There were five SEH releated patches posted in two threads in the last >> days. >> >> Two different patches in Martell e-mail starting this thread: the win64 >> seh (llvm) and the register names >> >> Three more related SEH patches in another thread: one for win64 seh >> clang, one for MinGW toolchain and another for unreachable prologue. >> >> To clarify and allow proper reviews for the different patches I opened >> reviews for four of them (the fifth got LGTM in the discussion but it does >> have the ownership issue you wrote) >> >> D3417 >> Emit a trap instruction for IR 'unreachable<http://reviews.llvm.org/D3417> >> >> D3418 >> SEH exceptions on Win64 (LLVM) <http://reviews.llvm.org/D3418> >> >> D3419 >> SEH exceptions on Win64 (clang part) <http://reviews.llvm.org/D3419> >> >> D3420 >> MinGW toolchain <http://reviews.llvm.org/D3420> >> >> >> Per your suggestion the link to the original discussion in D3418 was >> added. >> >> Code authors are: >> unreachable is by Vadim Chugunov. >> win64 seh llvm is by Kai Nacke + Martell re-posting. >> win64 seh clang is by Martell. >> mingw toolchain is by Martell. >> register names is by Ray Donnelly + Martell posting it. >> seh test for "register names" ( test is r206566) by myself. >> >> All but the "register names" patch were posted by their author on the >> llvm lists. >> >> Yaron >> >> >> >>-------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20140418/66c67e5e/attachment.html>