erichkeane added a comment. In D149647#4313311 <https://reviews.llvm.org/D149647#4313311>, @HazardyKnusperkeks wrote:
> But it is plain wrong. > It was done on purpose, so that e.g. `RemoveBracesLLVM` is not set when the > `SemiRemover` does its work. Ah, shoot, you're right! I missed that the 'return' was in a lambda! Thanks for the good catch! I think this patch is just 'wrong' in its entirety, and this is a case where Coverity needs to be suppressed. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D149647/new/ https://reviews.llvm.org/D149647 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits