bulbazord wrote: > LGTM! > > Should we `#include llvm/Support/Error.h"` in the BreakpointIDList header? Ah, I didn't add it but it builds so it's probably getting it transitively. I'll add it for future-proofing. https://github.com/llvm/llvm-project/pull/77858