https://github.com/ymand approved this pull request.
Thanks, looks good!
You can submit as is, but if you're up for it, it would actually be better to
add the new test case directly to the model's unittests. Something like this
test (though just one case is enough -- please put it in a separa
https://github.com/jcsxky updated
https://github.com/llvm/llvm-project/pull/68510
>From a5c5fc7a17f57a0b6ae328f7138435b4aaf7f9b5 Mon Sep 17 00:00:00 2001
From: huqizhi
Date: Sun, 8 Oct 2023 16:00:29 +0800
Subject: [PATCH] [clang][analysis]Use dyn_cast_or_null instead of cast to
prevent crash
jcsxky wrote:
> Thanks, looks good!
>
> You can submit as is, but if you're up for it, it would actually be better to
> add the new test case directly to the model's unittests. Something like this
> test (though just one case is enough -- please put it in a separate TEST_P):
>
> https://githu
https://github.com/jcsxky closed https://github.com/llvm/llvm-project/pull/68510
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits