[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