================
@@ -266,7 +265,7 @@ class RenameLocFinder : public 
RecursiveASTVisitor<RenameLocFinder> {
     return true;
   }
 
-  bool VisitDeclRefExpr(const DeclRefExpr *Expr) {
----------------
Sirraide wrote:

> How hard do you think it would be to introduce something like 
> `MutableDynamicRecursiveASTVisitor` and preserve constness where possible?

I mean, it would basically just involve copy-pasting the entire DRAV 
interface+implementation and adding `const` everywhere; personally, I don’t 
really think it’s worth it because there aren’t that many visitors that were 
using `const` in the first place, and the entire RAV implementation also isn’t 
`const` to begin with...

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

Reply via email to