Age | Commit message (Collapse) | Author |
|
As discussed in b4f6b26b5a1a78fecfa95ec2eb7ac8b80495d8aa "SAL_DLLPUBLIC_RTTI for
proper RTTI visibility for LLVM," RTTI-based -fsanitize= checks with Clang on
Linux need special precautions to make RTTI symbols visible across DSOs. The
approach taken there, as well as in 598d8194b0ea1a64e0ebba28a86c128bafa57c7c
"Visible function type RTTI for Clang -fsanitize=function," was to add explicit
SAL_DLLPUBLIC_RTTI annontations to relevant type definitions. However, for
-fsanitize=vptr that would have required many more of those, so it appears
easier to "misuse" -fsanitize-ms-compat in that case, which happens to give all
RTTI symbols default visibility (while otherwise still honoring our
SAL_DLLPUBLIC/PRIVATE annotations).
The SAL_DLLPUBLIC_RTTI annotations from 598d8194b0ea1a64e0ebba28a86c128bafa57c7c
"Visible function type RTTI for Clang -fsanitize=function" can likely be removed
again.
Change-Id: Ibeff7ab8c908111a7dc66ff0677204f112b24db8
|
|
Change-Id: I05923b7df753fdbc23f849c89f4f9f52a07915c7
|
|
Change-Id: I7701368e152ce20adbc05802c7c6518183919d1f
|
|
Change-Id: I3f50d3bb85bebdece692e069de73febc2ce63b47
|
|
...like 2fdf78109e815a64169fdab1a8175b63ef9c64d4 "Pass MAKE into
external/poppler's configure"
Change-Id: I4e2f1a13d120a7398fa81884710c589bb905714d
|
|
Change-Id: I8f2176500bf620cd5e4cdf434e6122b6163b3e0f
|
|
Change-Id: I9f60fd317f3a2995a182d51d06059bd994cf837c
|
|
Change-Id: I0523e49e640812be435ba4c97b1881ca253eb2ab
|
|
Change-Id: I48f67b203c3362ff87e41635e1212e0024a982f0
|
|
Change-Id: I04eda7844a5557f834f3ada2cef991b0a181c8a0
|
|
Change-Id: I9e1fc613816c943f4fb1033185e34e3acf317f1d
|
|
Change-Id: I0cc43cef91e3fcd82a3558a16ab0afbd4d56b141
|
|
Change-Id: I2fce6545d7f279e0e2d6f3ff53eee1ab82314135
|
|
Change-Id: Iffcc671ca41c5880579effe0786a3b4d3be0dab0
|
|
Change-Id: I69e87ad57513c7349aa0abf38b6054583d367963
|
|
Change-Id: Ib3e43e14f9223bc53f20f1241d9b8c0eaa5470ad
|
|
Change-Id: If26bdd6ed0be59a4fff137ac3166b9d644e1aea7
|
|
... why didn't GCC 4.1 have #pragma diagnostic push?
Change-Id: Iedb33d6451e46dc12e137bcd4dccd592c7771c23
|
|
Change-Id: I05d96875b9d1deb87dadd63f446080a6fe3494a9
Reviewed-on: https://gerrit.libreoffice.org/6630
Tested-by: LibreOffice gerrit bot <gerrit@libreoffice.org>
Reviewed-by: David Tardon <dtardon@redhat.com>
Tested-by: David Tardon <dtardon@redhat.com>
|