[PATCH] D48162: [GSoC] Schedule tree performance.
Siddharth via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sat Jun 16 12:15:03 PDT 2018
bollu added a comment.
Thanks for the patch! I've added some comments about general LLVM style, and some personal choices. Feel free to ignore the personal choices :)
- I value `const` highly, but this is not true across LLVM and Polly.
- Also, please run the file through `ninja polly-update-format`, It'll fix the formatting according to the formatting guidelines that are enforced using `clang-format`. I think in some places, the formatting is off?
https://reviews.llvm.org/D48162
More information about the llvm-commits
mailing list