jordan_rose added inline comments.
================ Comment at: lib/CodeGen/CGDebugInfo.cpp:2493-2499 case Type::Adjusted: - case Type::Decayed: + case Type::Decayed: { // Decayed and adjusted types use the adjusted type in LLVM and DWARF. - return CreateType( - cast<PointerType>(cast<AdjustedType>(Ty)->getAdjustedType()), Unit); + QualType Adjusted = cast<AdjustedType>(Ty)->getAdjustedType(); + (void)AttributedType::stripOuterNullability(Adjusted); + return CreateType(cast<PointerType>(Adjusted), Unit); + } ---------------- rsmith wrote: > I think this should be handled by `UnwrapTypeForDebugInfo` instead of here; > this is after all just a type sugar node. Getting back to this today. I'm inclined to say we should just drop the AdjustedType node altogether in UnwrapTypeForDebugInfo. What do you think? (also for @aprantl) Repository: rL LLVM https://reviews.llvm.org/D25850 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits