https://github.com/hanhanW commented: > // FIXME: Replace the uses of is(), get() and dyn_cast() with // isa, cast and the llvm::dyn_cast In this case, why do we replace it with `dyn_cast_if_present`, but not `dyn_cast`? https://github.com/llvm/llvm-project/pull/123693