https://github.com/michaelrj-google commented: This looks good, and I agree that tests aren't necessary. You do need to add `in_addr_t.h` to the dependencies in cmake though. https://github.com/llvm/llvm-project/pull/164830