On Wed, Feb 07, 2024 at 05:21:10PM +0800, Kewen.Lin wrote: > on 2024/2/6 14:01, Michael Meissner wrote: > Sorry for the possible confusion here, the "tune_proc" that I referred to is > the variable in the above else branch: > > enum processor_type tune_proc = (TARGET_POWERPC64 ? PROCESSOR_DEFAULT64 : > PROCESSOR_DEFAULT); > > It's either PROCESSOR_DEFAULT64 or PROCESSOR_DEFAULT, so it doesn't have a > chance to be PROCESSOR_FUTURE, so the checking "tune_proc == PROCESSOR_FUTURE" > is useless.
PROCESSOR_DEFAULT can be PROCESSOR_FUTURE if somebody configures GCC with --with-cpu=future. While in general it shouldn't occur, it is helpful to consider all of the corner cases. -- Michael Meissner, IBM PO Box 98, Ayer, Massachusetts, USA, 01432 email: meiss...@linux.ibm.com
