[clang] [clang][analyzer][NFC] Improve docs of alpha.unix.BlockInCriticalSection (PR #93812)

Endre Fülöp via cfe-commits cfe-commits at lists.llvm.org
Fri May 31 01:22:34 PDT 2024


gamesh411 wrote:

> What's the relationship between this PR and #93799 ?
> 
> Otherwise, the change LGTM, but you might want to either unify these two NFC changes into a single commit or ensure that they're independent.

Thanks for checking this!
To be honest, I started working on this, and then it hit me that a test case for the limitation would be nice 😸 
My intention was to ease the review as much as possible by cutting them into atomic parts.
They could indeed be independent; I will leave the limitation out of this and just introduce it in the other patch.


https://github.com/llvm/llvm-project/pull/93812


More information about the cfe-commits mailing list