[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