aaron.ballman added a comment.

In https://reviews.llvm.org/D48717#1147644, @JonasToth wrote:

> - fix decltype deduction with new matcher
>
>   I had to introduce a new matcher in clang for `DecltypeType` to reach its 
> underlying type. It would be better to have `hasType` resolve all these 
> issues but i dont know how much work that would be.


I don't think we want to modify `hasType()` -- that would strip off too much 
type information if it automatically reached through to the underlying type. 
However, we may want to consider adding something like `hasUnderlyingType()` 
that checks the matcher against each level of type sugar. I'm not certain this 
would be required for your patch, however.


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D48717



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

Reply via email to