This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH]: Power7 changes, part 2, add VSX registers, bswap*, address support, debugging


On Tue, Jun 23, 2009 at 05:21:43PM -0400, David Edelsohn wrote:
> Mike,
> 
> This patch is okay.
> 
> I forgot to mention in the first and second patches, there are some
> problems with indentation using spaces when they should be tabs.  In
> the second patch, it is rs6000_adjust_cost().  In the first patch, it
> is rs6000_override_options().

I fixed the rs6000_override_options.  I will fix rs6000_adjust cost shortly.

> As we discussed privately, hard_regno_mode_ok functions eventually
> should be updated to take TARGET_SINGLE_FLOAT and TARGET_DOUBLE_FLOAT
> into account.

Yep.

> Onward and upward...
> 
> Thanks, David

-- 
Michael Meissner, IBM
4 Technology Place Drive, MS 2203A, Westford, MA, 01886, USA
meissner@linux.vnet.ibm.com


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]