[Bug other/65911] [6 Regression] r222508 breaks clang-tblgen

2015-04-28 Thread tbsaunde at tbsaunde dot org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65911

--- Comment #6 from tbsaunde at tbsaunde dot org ---
On Tue, Apr 28, 2015 at 03:59:05AM +, trippels at gcc dot gnu.org wrote:
 https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65911
 
 Markus Trippelsdorf trippels at gcc dot gnu.org changed:
 
What|Removed |Added
 
  Status|UNCONFIRMED |NEW
Last reconfirmed||2015-04-28
  Ever confirmed|0   |1
 
 --- Comment #1 from Markus Trippelsdorf trippels at gcc dot gnu.org ---
 It's the ternary operator that causes the issue. 
 The following patch works fine:

huh, thanks for figuring it out!

Trev


[Bug ipa/61548] [5 Regression] FAIL: gcc.dg/tls/alias-1.c (internal compiler error)

2015-02-04 Thread tbsaunde at tbsaunde dot org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61548

--- Comment #20 from tbsaunde at tbsaunde dot org ---
 Created attachment 34673
   -- https://gcc.gnu.org/bugzilla/attachment.cgi?id=34673action=edit
 Proposed fix
 
 Ok, there seems to be 3 issues ;)  First is that emutls does not resolve alias
 after introducing it.  Second is that we may end up removing alias target
 before alias in ipa.c and third is that vaprool_node::remove actually removes
 the reference before attempting to use it.
 
 I would welcome if someone tests this on one of the targets reproducing the
 problem

Well, the patch I sent did 1 and 3, and that fixed the crash, so I
expect adding 2 won't really hurt, but I can test that theory tomorrow.

Trev