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]

PR middle-end/30028 (bcopy related testsuite failure)


Hi,
my patch for bcopy->memcpy conversion had amusing typo checking the
alignment wrong direction causing failure on some targets where memcpy
fails on overlapping targets.

Bootstrapped/regtested i686-linux, will commit it as obvious shortly.
My apologizes for the problem.
Honza

2006-11-30  Jan Hubicka  <jh@suse.cz>
	PR middle-end/30028
	* builtins.c (fold_builtin_memory_op): Fix typo in the check for
	memmove/bcopy->memcpy conversion.

Index: builtins.c
===================================================================
--- builtins.c	(revision 119360)
+++ builtins.c	(working copy)
@@ -8176,7 +8176,7 @@ fold_builtin_memory_op (tree arglist, tr
 	  if (dest_align && src_align
 	      && (readonly_data_expr (src)
 	          || (host_integerp (len, 1)
-		      && (MIN (src_align, dest_align) / BITS_PER_UNIT <=
+		      && (MIN (src_align, dest_align) / BITS_PER_UNIT >=
 			  tree_low_cst (len, 1)))))
 	    {
 	      tree fn = implicit_built_in_decls[BUILT_IN_MEMCPY];


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