aobolensk wrote: > The change looks fine, but I'm less sure about the include of `SelectionDAGNodes.h` in such a trivial header. Dropped completely unused ByteProvider template parameter and the header is gone now too https://github.com/llvm/llvm-project/pull/221959