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
  • [PATCH] D93214: [c... Jan Svoboda via Phabricator via cfe-commits
    • [PATCH] D9321... Duncan P. N. Exon Smith via Phabricator via cfe-commits
    • [PATCH] D9321... Jan Svoboda via Phabricator via cfe-commits

Reply via email to