[PATCH] D21488: [polly] Fix some Clang-tidy and Include What You Use warnings; other minor fixes
Tobias Grosser via llvm-commits
llvm-commits at lists.llvm.org
Mon Jun 20 19:25:13 PDT 2016
grosser added a comment.
[resending complete message]
The primary reason I ask to commit these changes separately is to educate people:
- why certain changes are beneficial and what they mean
- how these issues were found through automatic tooling.
Your answer indicates that you used a couple of independent tools to find these issues. It seems such changes are - besides being cleanups - independent and independently beneficial. Hence, it seems to make sense to apply them individually, also for people to see which change was performed by which tool. I assume a brief 2-3 sentence commit message that sets the context is enough for everybody to grasp the individual issues. Would it be possible for you to provide such kind of information without too much overhead?
Repository:
rL LLVM
http://reviews.llvm.org/D21488
More information about the llvm-commits
mailing list