[PATCH] D51329: [Attribute/Diagnostics] Print macro instead of whole attribute for address_space

Leonard Chan via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Aug 28 15:09:15 PDT 2018

leonardchan added inline comments.

Comment at: lib/AST/TypePrinter.cpp:1370
+    // Remove the underlying address space so it won't be printed.
+    SplitQualType SplitTy = T->getModifiedType().split();
rsmith wrote:
> This is unnecessary; just print the modified type here. (The modified type by definition does not have the attribute applied to it.)
When you say the modified type, do you mean just the type without it's qualifiers? I wasn't sure if removing all the qualifiers would suffice since there were also other  non-address_space qualifiers that could be printed.

  rC Clang


More information about the cfe-commits mailing list