[clang] [clang] Be const-correct with all uses of `Module *`. (PR #93493)
David Stone via cfe-commits
cfe-commits at lists.llvm.org
Tue May 28 10:06:51 PDT 2024
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
More information about the cfe-commits
mailing list