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, ARM] fix bootstrap on trunk


On Wed, 2009-11-18 at 13:55 -0600, Robert Nelson wrote:
> On Wed, Nov 18, 2009 at 11:36 AM, Richard Earnshaw <rearnsha@arm.com> wrote:
> >
> > On Tue, 2009-11-17 at 17:19 -0600, Robert Nelson wrote:
> >> On both my arm gcc builders (Sheevaplug & Beagleboard).  I've been
> >> getting this error on trunk while trying run the gcc bootstrap script:
> >
> > Your patch is missing a ChangeLog entry.
> >
> 
> Thanks, updated with a Changelog..
> 

Sorry for the delay replying (work, you know ... :-)

The ChangeLog wasn't quite suitable (take a look at the style of
existing entries and read the style guide for what they need to be), and
there was one more tiny change that I wanted to make, so I've committed
the following:

2009-11-25  Robert Nelson  <robert.nelson@digikey.com>
	    Richard Earnshaw  <rearnsha@arm.com>

	* arm.h (enum vfp_reg_type): Add VFP_NONE.
	* arm.c (all_fpus): Use it for floating point units that aren't based
	on the VFP architecture.
	(aapcs_vfp_is_call_or_return_candidate): Make base_mode a pointer to
	enum machine_mode.

*** config/arm/arm.c	(revision 154660)
--- config/arm/arm.c	(local)
*************** char arm_arch_name[] = "__ARM_ARCH_0UNK_
*** 821,830 ****
  
  static const struct arm_fpu_desc all_fpus[] =
  {
!   {"fpa",		ARM_FP_MODEL_FPA, 0, 0, false, false},
!   {"fpe2",		ARM_FP_MODEL_FPA, 2, 0, false, false},
!   {"fpe3",		ARM_FP_MODEL_FPA, 3, 0, false, false},
!   {"maverick",		ARM_FP_MODEL_MAVERICK, 0, 0, false, false},
    {"vfp",		ARM_FP_MODEL_VFP, 2, VFP_REG_D16, false, false},
    {"vfpv3",		ARM_FP_MODEL_VFP, 3, VFP_REG_D32, false, false},
    {"vfpv3-fp16",	ARM_FP_MODEL_VFP, 3, VFP_REG_D32, false, true},
--- 821,830 ----
  
  static const struct arm_fpu_desc all_fpus[] =
  {
!   {"fpa",		ARM_FP_MODEL_FPA, 0, VFP_NONE, false, false},
!   {"fpe2",		ARM_FP_MODEL_FPA, 2, VFP_NONE, false, false},
!   {"fpe3",		ARM_FP_MODEL_FPA, 3, VFP_NONE, false, false},
!   {"maverick",		ARM_FP_MODEL_MAVERICK, 0, VFP_NONE, false, false},
    {"vfp",		ARM_FP_MODEL_VFP, 2, VFP_REG_D16, false, false},
    {"vfpv3",		ARM_FP_MODEL_VFP, 3, VFP_REG_D32, false, false},
    {"vfpv3-fp16",	ARM_FP_MODEL_VFP, 3, VFP_REG_D32, false, true},
*************** use_vfp_abi (enum arm_pcs pcs_variant, b
*** 3853,3859 ****
  static bool
  aapcs_vfp_is_call_or_return_candidate (enum arm_pcs pcs_variant,
  				       enum machine_mode mode, const_tree type,
! 				       int *base_mode, int *count)
  {
    enum machine_mode new_mode = VOIDmode;
  
--- 3853,3859 ----
  static bool
  aapcs_vfp_is_call_or_return_candidate (enum arm_pcs pcs_variant,
  				       enum machine_mode mode, const_tree type,
! 				       enum machine_mode *base_mode, int *count)
  {
    enum machine_mode new_mode = VOIDmode;
  
*** config/arm/arm.h	(revision 154660)
--- config/arm/arm.h	(local)
*************** enum arm_fp_model
*** 307,312 ****
--- 307,313 ----
  
  enum vfp_reg_type
  {
+   VFP_NONE = 0,
    VFP_REG_D16,
    VFP_REG_D32,
    VFP_REG_SINGLE

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