AlexVlx added inline comments.
================
Comment at: lib/Transforms/IPO/FunctionImport.cpp:107
+static cl::opt<bool, true>
+ForceImportWeak("force-import-weak", cl::Hidden,
+ cl::desc("Allow weak functions to be imported"),
----------------
yaxunl wrote:
> Is it possible not to expose this option through extern? Generally these
> options should be kept static.
This seems to suggest that it is not utterly unacceptable:
http://llvm.org/docs/CommandLine.html#internal-vs-external-storage, and the use
case described there maps pretty closely to this, since the flag itself must be
accessible from two different TUs. Am I missing something?
Repository:
rL LLVM
https://reviews.llvm.org/D39739
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
