procyberian/gcc
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

h8300.c: Fix comment typos.

Browse source 
* h8300.c: Fix comment typos.
        (dosize): Declare the variable amount as unsigned.
        (get_shift_alg): Fix a comparison between signed and unsigned.
        (emit_a_shift): Likewise.
        (h8300_adjust_insn_length): Simplify the code.

From-SVN: r35609
This commit is contained in:
Kazu Hirata 2000-08-10 23:21:52 +00:00 • committed by Jeff Law
parent 6645c3fac8
commit 64530b8273
2 changed files with 16 additions and 10 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
gcc/ChangeLog
View file

@ -1,5 +1,11 @@
2000-08-10 Kazu Hirata <kazu@hxi.com>
* h8300.c: Fix comment typos.
(dosize): Declare the variable amount as unsigned.
(get_shift_alg): Fix a comparison between signed and unsigned.
(emit_a_shift): Likewise.
(h8300_adjust_insn_length): Simplify the code.
* c-decl.c: Fix formatting.
2000-08-10 Kaveh R. Ghazi <ghazi@caip.rutgers.edu>

20
gcc/config/h8300/h8300.c
View file

@ -160,7 +160,7 @@ dosize (file, op, size)
|| (TARGET_H8300 && current_function_needs_context
&& strcmp (op, "sub")))
{
HOST_WIDE_INT amount;
unsigned HOST_WIDE_INT amount;
/* Try different amounts in descending order. */
for (amount = (TARGET_H8300H || TARGET_H8300S) ? 4 : 2;
@ -1371,7 +1371,7 @@ print_operand_address (file, addr)
case CONST_INT:
{
/* Since the h8/300 only has 16 bit pointers, negative values are also
/* Since the H8/300 only has 16 bit pointers, negative values are also
those >= 32768. This happens for example with pointer minus a
constant. We don't want to turn (char *p - 2) into
(char *p + 65534) because loop unrolling can build upon this
@ -1725,8 +1725,8 @@ nshift_operator (x, mode)
}
/* Called from the .md file to emit code to do shifts.
Returns a boolean indicating success
(currently this is always TRUE). */
Return a boolean indicating success.
(Currently this is always TRUE). */
int
expand_a_shift (mode, code, operands)
@ -1973,7 +1973,7 @@ get_shift_alg (cpu, shift_type, mode, count, assembler_p,
/* We don't handle negative shifts or shifts greater than the word size,
they should have been handled already. */
if (count < 0 || count > GET_MODE_BITSIZE (mode))
if (count < 0 || (unsigned int) count > GET_MODE_BITSIZE (mode))
abort ();
switch (mode)
@ -2537,7 +2537,7 @@ emit_a_shift (insn, operands)
/* If the count is too big, truncate it.
ANSI says shifts of GET_MODE_BITSIZE are undefined - we choose to
do the intuitive thing. */
else if (n > GET_MODE_BITSIZE (mode))
else if ((unsigned int) n > GET_MODE_BITSIZE (mode))
n = GET_MODE_BITSIZE (mode);
alg = get_shift_alg (cpu_type, shift_type, mode, n, &assembler,
@ -2990,15 +2990,15 @@ h8300_adjust_insn_length (insn, length)
&& GET_MODE (SET_DEST (pat)) == SImode
&& INTVAL (SET_SRC (pat)) != 0)
{
int val = INTVAL (SET_SRC (pat));
if (TARGET_H8300
&& ((INTVAL (SET_SRC (pat)) & 0xffff) == 0
|| ((INTVAL (SET_SRC (pat)) >> 16) & 0xffff) == 0))
&& ((val & 0xffff) == 0
|| ((val >> 16) & 0xffff) == 0))
return -2;
if (TARGET_H8300H || TARGET_H8300S)
{
int val = INTVAL (SET_SRC (pat));
if (val == (val & 0xff)
|| val == (val & 0xff00))
return -6;