On 10/30/20 2:31 AM, Thomas Huth wrote: > Looking at the way the code is formatted here (there is an empty > line after break statements, but none where the break is missing), > the fallthrough is very likely intended here. So add a fallthrough > comment to make the it compilable with -Werror=implicit-fallthrough. > > Signed-off-by: Thomas Huth <th...@redhat.com> > --- > target/unicore32/translate.c | 2 ++ > 1 file changed, 2 insertions(+)
We don't need to guess based on blank lines. This matches with the instruction set overview at https://en.wikipedia.org/wiki/Unicore Reviewed-by: Richard Henderson <richard.hender...@linaro.org> r~
