legrosbuffle wrote:

> First what's a purpose of this change. Is there any other check that will 
> need it or not ?

We have a downstream check that wants to do additional processing after this 
one. We can't submit that check because it depends on a large piece of 
downstream infrastructure.

> With current implementation I don't see any "custom behaviour" that could be 
> implemented, as this act more like observer.

Indeed, the downstream check does not change the behaviour, it does extra stuff 
in addition what the check already does. I've rephrased the commit message to 
make that clearer.

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

Reply via email to