davidstone wrote: My philosophy on problems like that is that the first step to getting out of a hole is to stop digging the hole. We aren't going to make all of clang const-correct overnight, right? My goal here is starting with this one type: `Module`, and working on it. To me, this commit *is* considering the bigger picture. Rather than just saying "Oh, I am going to change the interface of `Module` so that it can control its invariants and to do that I need to make just these parts const-correct", I noticed I needed to make that change in some parts so as a preparatory commit (this one) I first make sure we are as const-correct as we can be with the current `Module` interface. Then I'll be going through and adding more const later on as I refactor more of `Module`.
https://github.com/llvm/llvm-project/pull/93493 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
