This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
RE: [PING] [PATCH] Fix parameters of __tsan_vptr_update
- From: Bernd Edlinger <bernd dot edlinger at hotmail dot de>
- To: Jakub Jelinek <jakub at redhat dot com>
- Cc: Mike Stump <mikestump at comcast dot net>, Konstantin Serebryany <konstantin dot s dot serebryany at gmail dot com>, Kostya Serebryany <kcc at google dot com>, Richard Biener <richard dot guenther at gmail dot com>, "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>, Dmitry Vyukov <dvyukov at google dot com>
- Date: Wed, 21 Jan 2015 12:49:39 +0100
- Subject: RE: [PING] [PATCH] Fix parameters of __tsan_vptr_update
- Authentication-results: sourceware.org; auth=none
- References: <DUB118-W519443EF06A1C4DB96059EE44F0 at phx dot gbl>,<CACT4Y+YBJRzG9c4KSrykO5NbaGeAKfmJUTb7ryqk4dX_Hx5aNQ at mail dot gmail dot com>,<DUB118-W2875D28C15EDA755AEEC50E44F0 at phx dot gbl>,<CACT4Y+a3j84+Pgp4zfuh2vRdDK3dy1WqA3oOMqhzWs+mQ5mqWg at mail dot gmail dot com>,<B45A3E4F-8313-45BE-946A-B0C159FF9120 at comcast dot net>,<CAGQ9bdxAVfod2cQUz2YQLxH9F1ovVkaqXC1qSO5JPfA9jPvWGg at mail dot gmail dot com>,<DUB118-W32DE3E55FB0ADCB62D2790E44B0 at phx dot gbl>,<CACT4Y+Z4jndiYsHuyjc6dGqtYLtrBLOpHZtniCU9TDuax-u4pQ at mail dot gmail dot com>,<DUB118-W33F7A757994E6FD8A01A76E4480 at phx dot gbl>,<DUB118-W3F97C81471F67C9945916E4480 at phx dot gbl>,<20150121113157 dot GN1746 at tucnak dot redhat dot com>
On Wed, 21 Jan 2015 12:31:57, Jakub Jelinek wrote:
>
> On Wed, Jan 21, 2015 at 12:17:51PM +0100, Bernd Edlinger wrote:
>>> test.c:3:6: internal compiler error: in expand_TSAN_FUNC_EXIT, at internal-fn.c:243
>>> void step (int i)
>>
>>
>> It looks like your patches shot each other down..
>>
>> How about this, maybe with a compile-time test of the step function in c-c++common/tsan?
>
> If you add a testcase for it, sure. It can be just compile time testcase,
> perhaps with some assembler match that no __tsan_ calls are made.
>
yes, that's what I had in mind.
Bernd.