On Sun, 2021-11-28 at 07:39 +0800, Xi Ruoyao via Gcc-patches wrote: > > > In function ‘int isa_fpu_compat_p(const loongarch_isa*, const > > loongarch_isa*)’, > > inlined from ‘void loongarch_config_target(loongarch_target*, long int, > > int, int, int, int, int, int, int)’ at > > ../../gcc/config/loongarch/loongarch-opts.c:194:28: > > ../../gcc/config/loongarch/loongarch-opts.c:436:24: error: > > ‘t.loongarch_target::isa.loongarch_isa::fpu’ may be used uninitialized > > [-Werror=maybe-uninitialized ] > > 436 | return set1 -> fpu == ISA_EXT_FPU32 || set1 -> fpu == > > ISA_EXT_FPU64; > > | ~~~~~~~~^~~
By the way, we shouldn't have whitespaces around "->" (i. e. it should be set1->fpu). > if (isa_fpu_compat_p (&(t.isa), &(force_isa))); And we shouldn't have those unneeded extra brackets, i. e. it should be &t.isa and &force_isa. -- Xi Ruoyao <xry...@mengyan1223.wang> School of Aerospace Science and Technology, Xidian University
