rjmccall wrote: IIRC, the original point of this function was to expose a way to get the canonical type extremely cheaply for operations that are just going to ignore local qualifiers anyway. The problem is not that that operation exists — it's good to use a cheaper operation when you can — it's that it's misnamed in a way that's encouraging misuse. We should probably break it into `getUnqualifiedCanonicalType`, which does the change you're proposing here, and `getCanonicalTypeWithoutLocalQualifiers()`, which hopefully has a sufficiently discouraging name that people won't accidentally use it.
https://github.com/llvm/llvm-project/pull/170271 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
