Hahnfeld added a comment. In https://reviews.llvm.org/D34888#799576, @gtbercea wrote:
> Does this also include the fixes in the following revision? > > https://reviews.llvm.org/D29905 Sorry, I wasn't aware of this revision and thought that it had long been committed. I just verified that the bug referenced in the summary is also fixed by my patch in https://reviews.llvm.org/D34888. However, I can't comment on whether this patch is still needed. Sorry for the conflicts if yes... You probably should commit your patches earlier, you currently have 10 accepted revisions that have not yet been committed. This will also avoid complicated rebases and so on. Repository: rL LLVM https://reviews.llvm.org/D29905 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits