thetruestblue wrote: This looks good. Makes sense now why forward declaring was the wrong thing here. My previous concerns we discussed are not an issue. https://github.com/llvm/llvm-project/pull/124351