On Tue, Oct 9, 2012 at 6:58 PM, Michael Meissner <meiss...@linux.vnet.ibm.com> wrote: > Ok, David preferred the 2 series of patches which replace all of the flags in > target_flags to rs6000_isa_flags to the 3 series of patches, which started > over, and added a new flag word, but did not change the existing options. > > In an effort to simplify the main patch, I'm going to push out some of the > patches that are standalone. This patch fixes the 3 signed/unsigned warnings > that were caused by comparing an integer type with an enumeration. I did > bootstap and make check with no regressions. Is it ok to install (it is > probably ok under the obvious rule)? > > 2012-10-09 Michael Meissner <meiss...@linux.vnet.ibm.com> > > * config/rs6000/rs6000.c (altivec_expand_dst_builtin): Fix signed > vs. unsigned warnings by using enum type for function code. > (paired_expand_builtin): Likewise. > (spe_expand_builtin): Likewise.
This patch is okay. Thanks, David