================
@@ -186,4 +218,370 @@ bool Sema::CheckCountedByAttrOnField(FieldDecl *FD, Expr 
*E, bool CountInBytes,
   return false;
 }
 
+SourceRange Sema::BoundsSafetySourceRangeFor(const CountAttributedType *CATy) {
----------------
Sirraide wrote:

> [#108631](https://github.com/llvm/llvm-project/pull/108631) seems to have 
> similar issues with wanting to access `AttributedTypeLoc` when all they have 
> is `AttributedType`. We might need to improve the infrastructure for 
> accessing `TypeLoc`s in a more ergonomic fashion.

Yeah, ideally, the attribute should really be moved from `AttributedTypeLoc` 
into `AttributedType` (Aaron and I had a discussion about this a few months 
back); this isn’t the first time we’ve run into problems because of that, but 
unfortunately, that wouldn’t even fix this because `CountAttributedType` is not 
an `AttributedType` (on that note, we have a lot of types that basically just 
implement an attribute; I wonder if there’d be a way to merge some of them and 
use just `AttributedType` instead once it actually stores the attribute, but 
that’s a different matter altogether ;Þ)

https://github.com/llvm/llvm-project/pull/106321
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to