Re: [PATCH] D22543: [libunwind] Properly align _Unwind_Exception.

2016-07-19 Thread Ed Maste via cfe-commits
emaste accepted this revision. emaste added a reviewer: emaste. emaste added a comment. This revision is now accepted and ready to land. I've committed the alignment change to the copy of llvm libunwind in the FreeBSD base system now. https://reviews.llvm.org/D22543 _

Re: [PATCH] D22543: [libunwind] Properly align _Unwind_Exception.

2016-07-19 Thread kled...@apple.com via cfe-commits
kledzik added a comment. LGTM https://reviews.llvm.org/D22543 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D22543: [libunwind] Properly align _Unwind_Exception.

2016-07-19 Thread Saleem Abdulrasool via cfe-commits
compnerd added a comment. LGTM; we should make sure Nick is okay with it as well (for Darwin). This should also make its way into 3.9 I think. https://reviews.llvm.org/D22543 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llv

[PATCH] D22543: [libunwind] Properly align _Unwind_Exception.

2016-07-19 Thread Eric Fiselier via cfe-commits
EricWF created this revision. EricWF added reviewers: mclow.lists, compnerd. EricWF added a subscriber: cfe-commits. _Unwind_Exception is required to be double word aligned. Currently the struct is under aligned. https://reviews.llvm.org/D22543 Files: include/unwind.h test/alignment.pass.cp