[PATCH] D13168: [OpenCL] OpenCL2.0 - Apply default address space rules
Anastasia Stulova via cfe-commits
cfe-commits at lists.llvm.org
Wed Oct 21 03:43:17 PDT 2015
Anastasia added inline comments.
================
Comment at: lib/Sema/SemaType.cpp:6282
@@ +6281,3 @@
+ } else if (state.getCurrentChunkIndex() == 0 &&
+ D.getContext() == Declarator::FileContext &&
+ !D.isFunctionDeclarator() && !D.isFunctionDefinition() &&
----------------
Anastasia wrote:
> pekka.jaaskelainen wrote:
> > Should this "white list" the cases where it's safe to add the default AS, not assume that it's applicable to any not listed here? Maybe safer that way.
> It seems like there are no corresponding helper methods available in Declarator. So the code is therefore using negation everywhere. But also if we would add a helper function, I think we would implement it as a form of negation anyways.
Any more thoughts here?
http://reviews.llvm.org/D13168
More information about the cfe-commits
mailing list