From 6de3644168854618eafd229d72356ead4f92a0cf Mon Sep 17 00:00:00 2001 From: Richard Sandiford Date: Sat, 19 May 2007 07:21:41 +0000 Subject: [PATCH] mips.c (mips_offset_within_alignment_p): Tweak comment. gcc/ * config/mips/mips.c (mips_offset_within_alignment_p): Tweak comment. Use a single return statement. From-SVN: r124853 --- gcc/ChangeLog | 5 +++++ gcc/config/mips/mips.c | 10 ++++------ 2 files changed, 9 insertions(+), 6 deletions(-) diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 78ea258a25f..4d75f782034 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,8 @@ +2007-05-19 Richard Sandiford + + * config/mips/mips.c (mips_offset_within_alignment_p): Tweak comment. + Use a single return statement. + 2007-05-19 Kaveh R. Ghazi PR middle-end/30250 diff --git a/gcc/config/mips/mips.c b/gcc/config/mips/mips.c index ff83eb7f4f9..f3f5d2b4538 100644 --- a/gcc/config/mips/mips.c +++ b/gcc/config/mips/mips.c @@ -1351,15 +1351,15 @@ mips_classify_symbol (rtx x) return SYMBOL_GENERAL; } -/* Returns true if OFFSET is within the range [0, ALIGN), where ALIGN +/* Return true if OFFSET is within the range [0, ALIGN), where ALIGN is the alignment (in bytes) of SYMBOL_REF X. */ static bool mips_offset_within_alignment_p (rtx x, HOST_WIDE_INT offset) { /* If for some reason we can't get the alignment for the - symbol, initializing this to one means we won't accept any - offset. */ + symbol, initializing this to one means we will only accept + a zero offset. */ HOST_WIDE_INT align = 1; tree t; @@ -1368,9 +1368,7 @@ mips_offset_within_alignment_p (rtx x, HOST_WIDE_INT offset) if (t) align = DECL_ALIGN_UNIT (t); - if (offset >= 0 && offset < align) - return true; - return false; + return offset >= 0 && offset < align; } /* Return true if X is a symbolic constant that can be calculated in -- 2.30.2