https://github.com/reactive-firewall commented: LGTM; with the already discussed caveat that there is room for future improvements. Specifically I like the additional comments for developers. Hope this helps https://github.com/llvm/llvm-project/pull/167998