dblaikie accepted this revision. dblaikie added a comment. This revision is now accepted and ready to land.
In D82728#2142071 <https://reviews.llvm.org/D82728#2142071>, @logan-5 wrote: > Feels like a dumb question, but I'm not sure if and how those build failures > are related to this patch? They seem to involve completely separate parts of > the LLVM project (and `.c` files to boot). Was it that the build was failing > for an unrelated reason at the moment the bots happened to build this patch? > Or am I misunderstanding something more obvious? Yeah, don't think those are related, no. Wouldn't worry about them. Looks good - thanks for the patch and all the details! Might be worth turning on by default in the LLVM build (after all the cleanup) Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D82728/new/ https://reviews.llvm.org/D82728 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits