[PATCH] D38816: Convert clang::LangAS to a strongly typed enum
Alexander Richardson via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Oct 13 08:39:32 PDT 2017
arichardson added inline comments.
================
Comment at: lib/AST/TypePrinter.cpp:1323
OS << "address_space(";
- OS << T->getEquivalentType().getAddressSpace();
+ OS << T->getEquivalentType()
+ .getQualifiers()
----------------
Anastasia wrote:
> arichardson wrote:
> > Anastasia wrote:
> > > Why do we need this change?
> > `__attribute__((address_space(n)))` is a target address space and not a language address space like `LangAS::opencl_generic`. Isn't `Qualifiers::getAddressSpaceAttributePrintValue()` meant exactly for this use case?
> Yes, I think there are some adjustment we do in this method to get the original source value to be printed corerctly. Does this mean we have no tests that caught this issue?
Seems like it, all tests pass both with and without this patch.
https://reviews.llvm.org/D38816
More information about the cfe-commits
mailing list