Hi all!

I found another issue of mklog.

Example:

--- a/gcc/asan.h
+++ b/gcc/asan.h
@@ -103,4 +103,14 @@ asan_intercepted_p (enum built_in_function fcode)
         || fcode == BUILT_IN_STRNCMP
         || fcode == BUILT_IN_STRNCPY;
 }
+
+/* Convert LEN to HOST_WIDE_INT if possible.
+   Returns -1 otherwise.  */
+
+static inline HOST_WIDE_INT
+maybe_tree_to_shwi (tree len)
+{
+  return tree_fits_shwi_p (len) ? tree_to_shwi (len) : -1;
+}
+
 #endif /* TREE_ASAN */

mklog output:

gcc/ChangeLog:

DATE

    * asan.h (asan_intercepted_p):
    (maybe_tree_to_shwi):

Currently mklog finds some changes for asan_intercepted_p which are do not exist.

Patched mklog output:

gcc/ChangeLog:

DATE

    * asan.h (maybe_tree_to_shwi):

Attached patch make mklog to stop search for changes inside function once '}' occur.

Ok, to commit?

--Marat
contrib/ChangeLog:

2014-11-06  Marat Zakirov  <m.zaki...@samsung.com>

	* mklog: Symbol '}' stops search for changes.  

diff --git a/contrib/mklog b/contrib/mklog
index 6ed4c6e..7de485d 100755
--- a/contrib/mklog
+++ b/contrib/mklog
@@ -117,7 +117,7 @@ sub is_top_level {
 	} else {
 		$function =~ s/^.//;
 	}
-	return $function && $function !~ /^[\s{}]/;
+	return $function && $function !~ /^[\s{]/;
 }
 
 # For every file in the .diff print all the function names in ChangeLog

Reply via email to