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

--- Comment #17 from gagan sidhu (broly) <broly at mac dot com> ---
(In reply to Xi Ruoyao from comment #16)
> (In reply to gagan sidhu (broly) from comment #15)
> 
> > and also: https://gcc.gnu.org/git/gitweb.cgi?p=gcc.git;h=9f943b2446f2d0
> 
> Please don't use this.  I've already said why this is not correct in
> previous replies.
> 
> The correct fix is https://reviews.llvm.org/D129749, which is not reviewed
> by upstream yet.  As libsanitizer is technically not a part of GCC we cannot
> apply a fix before it's accepted by upstream.

thank you for this.

after reading the replies and applying
https://github.com/gcc-mirror/gcc/commit/ee915c72da2caf92697dbedf0d9d9730ce9aca7a
i thought hans' fix would do the trick since it was building, but then it had a
problem on another ABI (i jumped the gun, my bad).

this led me to conclude the problem could be fixed with a size macro for the
64/N32 ABIs but i don't know the code very well.

i can confirm these two commits fix the problem:
https://gcc.gnu.org/git/gitweb.cgi?p=gcc.git;h=9f943b2446f2d0
https://reviews.llvm.org/D129749


thanks.

an aside:
not sure how much libsanitiser is used, and i suspect you're correct that i
could have done without it. 

not sure how all of these problems arose between 11.3 and 12.2. nice to see
things are back on track.

Reply via email to