[PATCH] D13468: [ELF2] -z now option implemented

Rui Ueyama via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 7 06:18:04 PDT 2015


ruiu accepted this revision.
ruiu added a comment.
This revision is now accepted and ready to land.

LGTM with a nit.


================
Comment at: ELF/Driver.cpp:139-143
@@ -138,1 +138,7 @@
 
+  for (auto *Arg : Args.filtered(OPT_z)) {
+    StringRef Opt = Arg->getValue();
+    if (Opt == "now")
+      Config->ZNow = true;
+  }
+
----------------
This can be

  for (auto *Arg : Args.filtered(OPT_z))
    if (Arg->getValue() == "now")
      Config->ZNow = true;



http://reviews.llvm.org/D13468





More information about the llvm-commits mailing list