foutrelis added a comment. In http://reviews.llvm.org/D12834#314136, @elizafox wrote:
> In http://reviews.llvm.org/D12834#311890, @foutrelis wrote: > > > We have received a few reports of clang crashes after applying the abi_tag > > support patch to our llvm/clang package in Arch Linux. > > > Why would you put a patch clearly marked as "needs review" into a > distribution?!?!?!?! We waited 6 months before switching to the new ABI in libstdc++. Ideally, we would have waited until the patch was reviewed and merged but did not want to wait much longer. I also (wrongly) considered the patch to be relative stable. > In any case, the recursion source seems obvious to me, but I don't know how > to add patches to this reviewboard item. If the correction is obvious as well and not very complex, would you mind sharing it? Repository: rL LLVM http://reviews.llvm.org/D12834 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
