jdoerfert added inline comments.
================ Comment at: llvm/lib/Frontend/OpenMP/OMPIRBuilder.cpp:5063 + } +} + ---------------- Style `F` ================ Comment at: llvm/lib/Frontend/OpenMP/OMPIRBuilder.cpp:5063 + } +} + ---------------- jdoerfert wrote: > Style `F` Why does this belong here? We should do it like clang, one function at a time, or in a generic place. This is not OpenMP specific at all. ================ Comment at: llvm/lib/IR/IRBuilder.cpp:1426 +} + IRBuilderDefaultInserter::~IRBuilderDefaultInserter() = default; ---------------- No llvm:: CHANGES SINCE LAST ACTION https://reviews.llvm.org/D147321/new/ https://reviews.llvm.org/D147321 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits