dexonsmith accepted this revision. dexonsmith added a comment. This revision is now accepted and ready to land.
This LGTM, just a whitespace request from `clang-format`. ================ Comment at: clang/lib/Frontend/InitPreprocessor.cpp:760-761 Builder.defineMacro("__SEH__"); - else if (LangOpts.DWARFExceptions && + else if (LangOpts.hasDWARFExceptions() && (TI.getTriple().isThumb() || TI.getTriple().isARM())) Builder.defineMacro("__ARM_DWARF_EH__"); ---------------- `clang-format` wants you to update this line. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D93214/new/ https://reviews.llvm.org/D93214 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits