[PATCH] D126137: [X86] Add support for `-mharden-sls=all`

Fangrui Song via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu May 26 13:57:55 PDT 2022


MaskRay requested changes to this revision.
MaskRay added inline comments.
This revision now requires changes to proceed.


================
Comment at: clang/lib/Driver/ToolChains/Arch/X86.cpp:257
+    } else if (Scope != "none") {
+      D.Diag(diag::err_invalid_sls_hardening) << Scope << A->getAsString(Args);
+    }
----------------
The convention is `err_drv_unsupported_option_argument`. Don't add a new kind.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D126137



More information about the cfe-commits mailing list