Anastasia added a comment.

> Can you explain why the changes to `TreeTransform` are required?

When address spaces of template parameter and argument are given explicitly 
they might mismatch. For example, `foo1` (in the test) is instantiated with  
`__local int` but the template parameter of `foo1` is  `__global T`. Currently 
clang fails to compile this with ICE while rebuilding the type that has 
multiple inconsistent addr space qualifiers. My patch fixes it, by giving an 
error instead of attempting to rebuild the type, that the addr space qualifiers 
mismatch.

There is another example of the same issue - `foo3`, but with non-pointer type.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D55127/new/

https://reviews.llvm.org/D55127



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to