Remove alignment value check in gasp

Nick Clifton nickc@cygnus.com
Tue Feb 29 17:16:00 GMT 2000


Hi Guys,

  Unless anyone objects I plan to check in the following patch
  tomorrow.  All it does is to remove the code in GASP that checks to
  see if the .align directive has been given an argument of 1, 2 or 4,
  This test is, I believe, bogus, since the .align directive takes a
  power-of-two as its argument and, for example ".align 3" is a valid
  alignment directive.

Cheers
	Nick


2000-02-29  Nick Clifton  <nickc@cygnus.com>

	* gasp.c (do_align): Remove bogus check of alignment value.

Index: gas/gasp.c
===================================================================
RCS file: /cvs/src//src/gas/gasp.c,v
retrieving revision 1.4
diff -p -r1.4 gasp.c
*** gasp.c	2000/01/26 22:48:31	1.4
--- gasp.c	2000/03/01 01:13:38
*************** do_align (idx, in)
*** 1439,1449 ****
        have_fill = 1;
      }
  
-   if (al != 1
-       && al != 2
-       && al != 4)
-     WARNING ((stderr, _("alignment must be one of 1, 2 or 4.\n")));
- 
    fprintf (outfile, ".align	%d", al);
    if (have_fill)
      fprintf (outfile, ",%d", fill);
--- 1439,1444 ----


More information about the Binutils mailing list