Hi!

Ok, provided it has been properly tested (given that we don't have
almost any testsuite so far, that is at least
rm -rf */{*/,}libsanitizer;
make # -jN
in the build tree and
make -C gcc check-g{cc,++} RUNTESTFLAGS='--target_board=unix\{-m32,-m64\} 
asan.exp'
).

On Fri, Nov 23, 2012 at 03:03:22PM +0400, Konstantin Serebryany wrote:
 2012-11-23  Kostya Serebryany  <k...@google.com>
         
+       * merge.sh: support tsan, support added/removed files.
+       * tsan/Makefile.am: remove deleted files.
+       * tsan/Makefile.in: ditto

ChangeLog entry description start with capital letter, so
Support tsan.  For tsan/Makefile.am it is better to be accurate,
        * tsan/Makefile.am (tsan_files): Remove tsan_printf.cc.
ditto is wrong (no dot at the end, no capital letter at the beginning,
but more importantly, as Makefile.in is a generated file, you want:
        * tsan/Makefile.in: Regenerated.
).

+       * other files: merge from upstream r168511. Note that HACKY_CALL in
+       tsan is disabled for now. See PR sanitizer/55376.

Just mention
        Merge from upstream r168511
above the merge.sh line, ChangeLog entry shouldn't contain notes etc.
+
+2012-11-23  Kostya Serebryany  <k...@google.com>
+        
        * merge.sh: a script that will help merges from upstream.

This should have been
        * merge.sh: New file.
 

        Jakub

Reply via email to