[PATCH] D26340: [analyzer] Add SpinLockChecker for the Magenta kernel

Kareem Khazem via cfe-commits cfe-commits at lists.llvm.org
Wed Nov 9 12:30:43 PST 2016


khazem marked 4 inline comments as done.
khazem added a comment.

Devin, based on Artem's review of the other checker that I have posted [1] I am wondering about merging both this SpinLockChecker and the MutexChecker into PthreadLockChecker. Do you think it is still worth landing this SpinLockChecker, or do you suppose that abandoning this patch and working on merging all the checkers would be a good idea? I've noted how the checkers might be merged in my reply to Artem: [2].

[1] https://reviews.llvm.org/D26342
[2] https://reviews.llvm.org/D26342#590881


https://reviews.llvm.org/D26340





More information about the cfe-commits mailing list