https://github.com/clayborg commented: IIUC this is just moving funtionality around right now so that we can eventually plug-in a different `IndexSet<T>` where T is an improved version that doesn't use `ConstString` objects? https://github.com/llvm/llvm-project/pull/123732