PR lto/51916
* lto-object.c (LTO_SEGMENT_NAME): Define segment name.
(lto_obj_file_open): Use it.
Patrick, the changelog was referring to the wrong file and the wrong
function. I've fixed it. I also inlined the segment name, as Ian had
in the PR.
Richard Guenther approved the patch on
Dominique or Iain, may I ask you to test this patch on darwin? I have a
sporadic access to a darwin machine.
The patch fixes the PR without regression on x86_64-apple-darwin10.
Bootstraping on powerpc-apple-darwin9 will finish in a couple hours.
Thanks for the debugging and the patch,
Hi Patrick,
thanks for doing this,
On 21 Jan 2012, at 22:57, Patrick Marlier wrote:
Dominique or Iain, may I ask you to test this patch on darwin? I
have a sporadic access to a darwin machine.
bootstrapped on i686-darwin9 - tests in progress...
minor observations:
PR lto/51916
On 01/20/2012 08:04 AM, Dominique Dhumieres wrote:
The following patch fixes the problem.
The test fails on *-apple-darwin*: pr51916.
TIA
Dominique
Dominique or Iain, may I ask you to test this patch on darwin? I have a
sporadic access to a darwin machine.
Bootstrapped and tested on
On 01/20/2012 08:04 AM, Dominique Dhumieres wrote:
The following patch fixes the problem.
The test fails on *-apple-darwin*: pr51916.
TIA
Dominique
Dominique or Iain, may I ask you to test this patch on darwin? I have a
sporadic access to a darwin machine.
Bootstrapped and tested on
The following patch fixes the problem.
The test fails on *-apple-darwin*: pr51916.
TIA
Dominique
On Wed, 18 Jan 2012, Aldy Hernandez wrote:
On 01/18/12 03:09, Richard Guenther wrote:
On Tue, 17 Jan 2012, Aldy Hernandez wrote:
What I have in mind is to abstract out the initialization of TM
builtins
from
gtm-builtins.def (through its inclusion in builtins.def)
On Tue, 17 Jan 2012, Aldy Hernandez wrote:
What I have in mind is to abstract out the initialization of TM builtins
from
gtm-builtins.def (through its inclusion in builtins.def) into a separate
function that we can call either in c_define_builtins() from the C-ish
front-ends, or
On 01/18/12 03:09, Richard Guenther wrote:
On Tue, 17 Jan 2012, Aldy Hernandez wrote:
What I have in mind is to abstract out the initialization of TM builtins
from
gtm-builtins.def (through its inclusion in builtins.def) into a separate
function that we can call either in c_define_builtins()
On 12/20/11 03:43, Richard Guenther wrote:
On Mon, 19 Dec 2011, Patrick Marlier wrote:
On 12/16/2011 03:54 AM, Richard Guenther wrote:
On Thu, 15 Dec 2011, Patrick Marlier wrote:
In PR51280, LTO does ICE because the object file uses TM builtin but the
TM is
not enabled.
In the patch, it
On Tue, 17 Jan 2012, Aldy Hernandez wrote:
On 12/20/11 03:43, Richard Guenther wrote:
On Mon, 19 Dec 2011, Patrick Marlier wrote:
On 12/16/2011 03:54 AM, Richard Guenther wrote:
On Thu, 15 Dec 2011, Patrick Marlier wrote:
In PR51280, LTO does ICE because the object file
What I have in mind is to abstract out the initialization of TM builtins from
gtm-builtins.def (through its inclusion in builtins.def) into a separate
function that we can call either in c_define_builtins() from the C-ish
front-ends, or from lto_define_builtins (once we encounter a TM builtin
On Mon, 19 Dec 2011, Patrick Marlier wrote:
On 12/16/2011 03:54 AM, Richard Guenther wrote:
On Thu, 15 Dec 2011, Patrick Marlier wrote:
In PR51280, LTO does ICE because the object file uses TM builtin but the
TM is
not enabled.
In the patch, it displays a error message if the
On 12/16/2011 03:54 AM, Richard Guenther wrote:
On Thu, 15 Dec 2011, Patrick Marlier wrote:
In PR51280, LTO does ICE because the object file uses TM builtin but the TM is
not enabled.
In the patch, it displays a error message if the builtin is not defined and
due to TM.
I moved is_tm_builtin()
In PR51280, LTO does ICE because the object file uses TM builtin but the
TM is not enabled.
In the patch, it displays a error message if the builtin is not defined
and due to TM.
I moved is_tm_builtin() from calls.c to trans-mem.c. I splitted it into
2 functions
15 matches
Mail list logo