[PATCH] D129886: [clang] Add -fdiagnostics-format=sarif option for future SARIF output

[Vaibhav Yenamandra via Phabricator via cfe-commits]

vaibhav.y added a comment.

Please address the whitespace changes in `clang/lib/Driver/ToolChains/Clang.cpp`.



================
Comment at: clang/lib/Driver/ToolChains/Clang.cpp:80-81
             Args.getLastArg(options::OPT_dynamic, options::OPT_mdynamic_no_pic))
-      D.Diag(diag::err_drv_argument_not_allowed_with) << A->getAsString(Args)
-                                                      << "-static";
+      D.Diag(diag::err_drv_argument_not_allowed_with)
+          << A->getAsString(Args) << "-static";
 }
----------------
nit: Looks like this an unrelated format change/ rebase issue. There are a few others as well in this file

Would be better to introduce these through another CR, or use `git clang-format <commit-start> [<commit-end>]` to limit it to the diff.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D129886/new/

https://reviews.llvm.org/D129886