Mercurial > hg > CbC > CbC_gcc
comparison libsanitizer/HOWTO_MERGE @ 111:04ced10e8804
gcc 7
author | kono |
---|---|
date | Fri, 27 Oct 2017 22:46:09 +0900 |
parents | |
children | 1830386684a0 |
comparison
equal
deleted
inserted
replaced
68:561a7518be6b | 111:04ced10e8804 |
---|---|
1 In general, merging process should not be very difficult, but we need to | |
2 track various ABI changes and GCC-specific patches carefully. Here is a | |
3 general list of actions required to perform the merge: | |
4 | |
5 * Checkout recent GCC tree. | |
6 * Run merge.sh script from libsanitizer directory. | |
7 * Modify Makefile.am files into asan/tsan/lsan/ubsan/sanitizer_common/interception | |
8 directories if needed. In particular, you may need to add new source files | |
9 and remove old ones in source files list, add new flags to {C, CXX}FLAGS if | |
10 needed and update DEFS with new defined variables. You can find these changes | |
11 in corresponding CMakeLists.txt and config-ix.cmake files from compiler-rt source | |
12 directory. | |
13 * Apply all needed GCC-specific patches to libsanitizer (note that some of | |
14 them might be already included to upstream). The list of these patches is stored | |
15 into LOCAL_PATCHES file. | |
16 * Apply all necessary compiler changes. Be especially careful here, you must | |
17 not break ABI between compiler and library. You can reveal these changes by | |
18 inspecting the history of AddressSanitizer.cpp and ThreadSanitizer.cpp files | |
19 from LLVM source tree. | |
20 * Update ASan testsuite with corresponding tests from lib/asan/tests directory. | |
21 Not all tests can be migrated easily, so you don't need them all to be adapted. | |
22 * Modify configure.ac file if needed (e.g. if you need to add link against new | |
23 library for sanitizer lilbs). | |
24 * Add new target platforms in configure.tgt script if needed. | |
25 * Bump SONAME for sanitizer libraries in asan/tsan/ubsan libtool-version files | |
26 if ABI has changed. | |
27 * Regenerate configure script and all Makefiles by autoreconf. You should use | |
28 exactly the same autoconf and automake versions as for other GCC directories (current | |
29 versions are written in Makefile.in and configure files). | |
30 * Run regression testing on at least three platforms (e.g. x86-linux-gnu, x86_64-linux-gnu, | |
31 aarch64-linux-gnu, arm-linux-gnueabi). | |
32 * Run {A, UB}San bootstrap on at least three platforms. | |
33 * Compare ABI of corresponding libclang_rt-asan and newly build libasan libraries. | |
34 You can use a pretty good libabigail tool (https://sourceware.org/libabigail/index.html) | |
35 to perform such a comparision. Note, that the list of exported symbols may differ, | |
36 e.g. because libasan currently does not include UBSan runtime. | |
37 * Split your changes into logical parts (e.g. raw merge, compiler changes, GCC-specific changes | |
38 in libasan, configure/Makefile changes). The review process has O(N^2) complexity, so you | |
39 would simplify and probably speed up the review process by doing this. | |
40 * Send your patches for review to GCC Patches Mailing List (gcc-patches@gcc.gnu.org). | |
41 * Update LOCAL_PATCHES file when you've committed the whole patch set with new revisions numbers. |