bruno accepted this revision. bruno added a comment. This revision is now accepted and ready to land.
LGTM, it would be great if someone else stamps this too, in case I missed something. ================ Comment at: clang/include/clang/Driver/Options.td:164 +def m_m68k_Features_Group: OptionGroup<"<m68k features group>">, + Group<m_Group>, DocName<"M68k">; def m_mips_Features_Group : OptionGroup<"<mips features group>">, ---------------- rengolin wrote: > craig.topper wrote: > > bruno wrote: > > > Looks like this is missing `clang-format`? > > I don't think clang-format really understands tablegen files. > It does not. Don't clang-format table-gen files, the end result is > horrendous. :) Thanks for pointing out, thinko for "please format" CHANGES SINCE LAST ACTION https://reviews.llvm.org/D88394/new/ https://reviews.llvm.org/D88394 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits