[PATCH] D13048: Fix for merging decls in pragma weak

Aaron Ballman via cfe-commits cfe-commits at lists.llvm.org
Tue Sep 22 06:01:34 PDT 2015


aaron.ballman added inline comments.

================
Comment at: lib/Sema/SemaDeclAttr.cpp:5201
@@ -5200,5 +5200,3 @@
   if (FunctionDecl *FD = dyn_cast<FunctionDecl>(ND)) {
-    FunctionDecl *NewFD;
-    // FIXME: Missing call to CheckFunctionDeclaration().
     // FIXME: Mangling?
     // FIXME: Is the qualifier info correct?
----------------
If we're in this code, any chance you can also look at the other FIXME statements? If not, that's fine -- this is still positive movement. But it's usually best to tackle all of these at once while it's fresh in someone's mind what's happening here.


http://reviews.llvm.org/D13048





More information about the cfe-commits mailing list