[clang] [clang] Add visibility to AST dump (PR #218113)

[Helena Kotas via cfe-commits]

================
@@ -1477,6 +1477,28 @@ void TextNodeDumper::dumpFormalLinkage(const NamedDecl *ND) {
   case Linkage::VisibleNone:
     llvm_unreachable("Not a formal linkage!");
   }
+
+  // FIXME: HLSLAttributedResourceType should always have contained type,
----------------
hekota wrote:

I am not very familiar with linkage calculations, but if the calculations are queued off an inner type, the `HLSLAttributesResourceType` has a wrapped type (`getWrappedType()`) which is always non-null and should always be the builtin type `__hlsl_resource_t`.  

https://github.com/llvm/llvm-project/pull/218113