[PATCH] D81404: [AArch64] Add clang command line support for -mharden-sls=

Kristof Beyls via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jun 19 02:07:27 PDT 2020


kristof.beyls marked 2 inline comments as done.
kristof.beyls added inline comments.


================
Comment at: clang/lib/Driver/ToolChains/Arch/AArch64.cpp:229
+      Scope.split(Opts, ",");
+      for (int I = 0, E = Opts.size(); I != E; ++I) {
+        StringRef Opt = Opts[I].trim();
----------------
ostannard wrote:
> Could this be a range-based for loop?
Yep, thanks for the suggestion. I've changed that before committing this.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D81404





More information about the cfe-commits mailing list