[PATCH] D12903: Add -fplugin=name.so option to the driver

Saleem Abdulrasool via cfe-commits cfe-commits at lists.llvm.org
Fri Sep 18 20:22:05 PDT 2015


compnerd accepted this revision.
compnerd added a reviewer: compnerd.
compnerd added a comment.
This revision is now accepted and ready to land.

Looks fine minus the one thing that needs a quick look at.


================
Comment at: include/clang/Driver/Options.td:952
@@ -951,1 +951,3 @@
 def fno_pie : Flag<["-"], "fno-pie">, Group<f_Group>;
+def fplugin : Joined<["-"], "fplugin=">, Group<f_Group>, Flags<[DriverOption]>, MetaVarName<"<dsopath>">,
+  HelpText<"Load the named plugin (dynamic shared object)">;
----------------
Don't most options of this type use the _EQ suffix?  Might be nice to follow the convention.


Repository:
  rL LLVM

http://reviews.llvm.org/D12903





More information about the cfe-commits mailing list