vsapsai added inline comments.
================ Comment at: clang/lib/Sema/SemaDeclAttr.cpp:7207-7217 + } else { + FixIts.push_back(FixItHint::CreateInsertion( + SelectorLocs[I], SelectorSlotNames[I])); + } + } + } else { + FixIts.push_back(FixItHint::CreateReplacement(UseRange, Replacement)); ---------------- aaron.ballman wrote: > You can elide a lot of braces here. Just to clarify, is it OK if if-branch is with braces but else-branch without? I.e. ``` if (...) { line 1 line 2 } else the only line ``` https://reviews.llvm.org/D44589 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits