https://github.com/Sirraide commented:

Seems fine, though I wonder at what point we might want to rewrite this to be a 
switch statement on the StmtClass because we’re checking for like 5 different 
expressions here at this point

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

Reply via email to