[PATCH] D121169: Cleanup codegen includes

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 8 02:49:19 PST 2022


RKSimon accepted this revision.
RKSimon added a comment.
This revision is now accepted and ready to land.

In D121169#3366678 <https://reviews.llvm.org/D121169#3366678>, @foad wrote:

> In D121169#3366677 <https://reviews.llvm.org/D121169#3366677>, @lenary wrote:
>
>> This is part of the work described here: https://discourse.llvm.org/t/include-what-you-use-include-cleanup/5831
>>
>> There have been quite a few of these patches over the few weeks, each addressing a different part of LLVM.
>
> Thanks. I still think the commit message should say a bit more than "cleanup".

Yes - please ensure your summary references the https://discourse.llvm.org/t/include-what-you-use-include-cleanup/5831 conversation URL

LGTM


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D121169/new/

https://reviews.llvm.org/D121169



More information about the llvm-commits mailing list