jcownie added a comment.
Good point. My personal preference would be to use "== 0", rather than logical not, since I find that easier to read.
So
if (((buf.edx >> 9) & 1) == 0) {
...
}
But your fix is fine too.
https://reviews.llvm.org/D28599