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