On Thu, Apr 21, 2011 at 03:02:10PM -0400, Michael Meissner wrote: > In looking at some improvements to the powerpc, we wanted to change the > default > for when a table jump is generated vs. a series of if statements. Now, we > could just add a powerpc specific TARGET_CASE_VALUES_THRESHOLD, but I tend to > think that these should be settable on all/most ports with --param. > > At present, there are only two ports (avr and mn10300) that define their own > TARGET_CASE_VALUES_THRESHOLD hook. My first patch does not remove the target > hook and modify the avr/mn10300 ports to use maybe_set_param_value, but that > can be done if desired. > > The patch adds two --param values, one for when the port is using the casesi > insn, and the other when it uses the more primitive tablejump insn. > > I have bootstrapped the compiler with this patch and run the test suite with > no > regressions. Is it ok to apply as is? Should I modify the avr and mn10300 > ports to use the parameters and do away with the target hook? Or should I do > this just as a powerpc target hook?
I never got a response for this, and my earlier ping didn't seem to go out. I'll check it in on Monday if there are no objections. -- Michael Meissner, IBM 5 Technology Place Drive, M/S 2757, Westford, MA 01886-3141, USA meiss...@linux.vnet.ibm.com fax +1 (978) 399-6899