[PATCH] D39739: [HCC] Add flag to Import Weak Functions in Function Importer

[Alex Voicu via Phabricator via cfe-commits]

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