[PATCH] D105169: [Clang/Test]: Rename enable_noundef_analysis to disable-noundef-analysis and turn it off by default
Hyeongyu Kim via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Dec 28 19:10:40 PST 2021
hyeongyukim added a comment.
In D105169#3211929 <https://reviews.llvm.org/D105169#3211929>, @nathanchance wrote:
> @hyeongyukim I am currently offline for the evening but it seems like my reduction might have been too aggressive. It looks like this code comes from `ravb_set_gti()` (https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/drivers/net/ethernet/renesas/ravb_main.c?h=v5.16-rc7#n2480), which checks that `rate` is not zero before using it as a divisor. I will see if I can get a reproducer without any undefined behavior such as this.
Okay! Thank you for your response.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D105169/new/
https://reviews.llvm.org/D105169
More information about the cfe-commits
mailing list