Steelskin wrote: > > Should I apply the diff [the bot is talking > > about](https://github.com/llvm/llvm-project/pull/174892#issuecomment-3740884157)? > > Note that I didn't touch those files. > > I would ignore the bot, that seems like a false positive; CC @tstellar > @petrhosek as infra folks
This is not a false positive in the sense that these headers are included by the header you are modifying `llvm/include/llvm/IR/DiagnosticInfo.h` and are missing the annotations. However, your PR did not cause the issue so feel free to ignore it. I will land the fix separately, thanks for tagging me! https://github.com/llvm/llvm-project/pull/174892 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
