Eugene.Zelenko added inline comments.
================ Comment at: clang-tidy/fpga/StructPackAlignCheck.h:1 +//===--- StructPackAlignCheck.h - clang-tidy---------------------*- C++ -*-===// +// ---------------- Please add space after clang-tidy. Same in source file. ================ Comment at: docs/clang-tidy/checks/fpga-struct-pack-align.rst:12 + +Based on the "Altera SDK for OpenCL: Best Practices Guide" found here: +https://www.altera.com/en_US/pdfs/literature/hb/opencl-sdk/aocl_optimization_guide.pdf ---------------- You could use link syntax. See Fuchsia checks as example. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D66564/new/ https://reviews.llvm.org/D66564 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits