zahiraam wrote:

> idxcontents is only used when we have a set of file names available. hence 
> this was only set via relevant constructors and rest of the runtime doesn't 
> use the field.
> 
> but we don't construct tons of new index instances every second. so it's 
> totally fine to just initialize them to `none`

Thanks for the explanation.

https://github.com/llvm/llvm-project/pull/118324
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to