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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D81404: [... Kristof Beyls via Phabricator via cfe-commits
    • [PATCH] D814... Oliver Stannard (Linaro) via Phabricator via cfe-commits
    • [PATCH] D814... Kristof Beyls via Phabricator via cfe-commits
    • [PATCH] D814... Kristof Beyls via Phabricator via cfe-commits

Reply via email to