[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