[Bug sanitizer/88684] Please make SANITIZER_NON_UNIQUE_TYPEINFO a runtime flag (or always true)

[jakub at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88684

Jakub Jelinek <jakub at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|9.0                         |---
            Summary|[7/8/9 Regression] Please   |Please make
                   |make                        |SANITIZER_NON_UNIQUE_TYPEIN
                   |SANITIZER_NON_UNIQUE_TYPEIN |FO a runtime flag (or
                   |FO a runtime flag (or       |always true)
                   |always true)                |

--- Comment #9 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
Why is this actually considered a regression?
While in GCC 6 and older libsanitizer didn't have SANITIZER_NON_UNIQUE_TYPEINFO
macro, it performed only == comparisons of the string pointers, so in the end
it acted as if the current default of SANITIZER_NON_UNIQUE_TYPEINFO.  Not
suitable for libstdc++, sure, but not a regression.

That said, I'm willing to ack it for GCC9 even then if upstream comes up with
something or if they don't care, eventually as a GCC only tweak.