[clang] [clang] Be const-correct with all uses of `Module *`. (PR #93493)
Matheus Izvekov via cfe-commits
cfe-commits at lists.llvm.org
Tue May 28 11:40:58 PDT 2024
mizvekov wrote:
> `const_cast` here is a relatively recent addition, and I checked out with @erichkeane that the use case (pack expansion) is legit. According to the approach you're suggesting, the one who wrote this `const_cast` should instead refactor the MLTAL to use `llvm::MutableArrayRef`.
Take a guess who that person was :grin:
> I tried, pretty hard, but ultimately failed. The change turned out to be touching every corner of our template engine, with no end in sight.
Oh well, sorry I wasn't around to advise against it.
https://github.com/llvm/llvm-project/pull/93493
More information about the cfe-commits
mailing list