=?utf-8?q?Hervé?= Poussineau <hpous...@reactos.org>, =?utf-8?q?Hervé?= Poussineau <hpous...@reactos.org>, =?utf-8?q?Hervé?= Poussineau <hpous...@reactos.org>, =?utf-8?q?Hervé?= Poussineau <hpous...@reactos.org>, =?utf-8?q?Hervé?= Poussineau <hpous...@reactos.org>, =?utf-8?q?Hervé?= Poussineau <hpous...@reactos.org>, =?utf-8?q?Hervé?= Poussineau <hpous...@reactos.org>, =?utf-8?q?Hervé?= Poussineau <hpous...@reactos.org>, =?utf-8?q?Hervé?= Poussineau <hpous...@reactos.org>, =?utf-8?q?Hervé?= Poussineau <hpous...@reactos.org>, =?utf-8?q?Hervé?= Poussineau <hpous...@reactos.org>, =?utf-8?q?Hervé?= Poussineau <hpous...@reactos.org> Message-ID: In-Reply-To: <llvm.org/llvm/llvm-project/pull/107...@github.com>
hpoussin wrote: > I browsed through the first couple commits and added a couple pointers. > Overall it looks reasonable, but things can probably be split up into even > smaller commits for atomicity, which also would make it clearer where we'd > want more test coverage. Thanks for the review. I'll take it into account when creating PRs for each part. https://github.com/llvm/llvm-project/pull/107744 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
