arphaman added inline comments.

================
Comment at: lib/CodeGen/CGObjC.cpp:3435
+      CheckFTy, "__clang_at_available_requires_core_foundation_framework"));
+  CFLinkCheckFunc->setLinkage(llvm::GlobalValue::LinkOnceAnyLinkage);
+  CodeGenFunction CGF(*this);
----------------
rjmccall wrote:
> Is this a public weak symbol?  Make it hidden, please.
> 
> Did you consider just adding "-framework CoreFoundation"  to the module link 
> options?
I'm not familiar with module link options, I'll look into them.


Repository:
  rL LLVM

https://reviews.llvm.org/D30977



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to