This is true of all the other types as well, though; debug info has many known holes, I don't think we need a FIXME. The good news is that I believe we are going to get some help to polish this off soon!
- Daniel On Fri, Feb 27, 2009 at 3:18 PM, Douglas Gregor <[email protected]> wrote: > > On Feb 27, 2009, at 3:15 PM, Eli Friedman wrote: > >> Author: efriedma >> Date: Fri Feb 27 17:15:07 2009 >> New Revision: 65659 >> >> URL: http://llvm.org/viewvc/llvm-project?rev=65659&view=rev >> Log: >> Fix enumeration in switch warnings. No behavior change. >> >> >> Modified: >> cfe/trunk/lib/CodeGen/CGDebugInfo.cpp >> >> Modified: cfe/trunk/lib/CodeGen/CGDebugInfo.cpp >> URL: >> http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/CodeGen/CGDebugInfo.cpp?rev=65659&r1=65658&r2=65659&view=diff >> >> = >> = >> = >> = >> = >> = >> = >> = >> ====================================================================== >> --- cfe/trunk/lib/CodeGen/CGDebugInfo.cpp (original) >> +++ cfe/trunk/lib/CodeGen/CGDebugInfo.cpp Fri Feb 27 17:15:07 2009 >> @@ -496,6 +496,12 @@ >> case Type::ObjCInterface: >> case Type::ObjCQualifiedInterface: >> case Type::ObjCQualifiedId: >> + case Type::FixedWidthInt: >> + case Type::BlockPointer: >> + case Type::MemberPointer: >> + case Type::ClassTemplateSpecialization: >> + case Type::ObjCQualifiedClass: >> + // Unsupported types >> return llvm::DIType(); >> >> case Type::Builtin: Slot = CreateType(cast<BuiltinType>(Ty), >> Unit); break; > > Please at least put a FIXME in there, because we will eventually need > to support them. > > - Doug > _______________________________________________ > cfe-commits mailing list > [email protected] > http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits > _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
