Szelethus added a comment.

>> checker callback evaluation order is ensured
> 
> And it will be the same for all callbacks, right? Like, we're not doing this 
> thing yet where it intentionally goes like
> 
>   chk1->checkPreCall();
>   chk2->checkPreCall();
>   chk2->checkPostCall();
>   chk1->checkPostCall();
> 
> 
> ?

It should work for all callbacks. It'd be a nightmare to test and `assert`, but 
when `CheckerManager` registers a checker, all of its callbacks are registered 
to the appropriate container, which isn't sorted.

> NoQ added a reviewer: vsavchenko.

Woops O:)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D80905



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

Reply via email to