diff mbox

Fix vms targets

Message ID 5636F464.8090409@redhat.com
State New
Headers show

Commit Message

Jeff Law Nov. 2, 2015, 5:28 a.m. UTC
The header reduction didn't seem to handle the vms targets correctly. 
This reverts that part of Andrew's patch which allows the alpha, alpha64 
and ia64 vms targets to build again.

Installed on the trunk.  That covers all the fallout from standard 
builds of config-list.mk that I'm aware of.



Jeff
commit 1eddb04c7103a2f87118fc98111c1877b544a6a9
Author: law <law@138bc75d-0d04-0410-961f-82ee72b054a4>
Date:   Mon Nov 2 05:27:14 2015 +0000

    [PATCH] Fix vms targets
    
    	* vmsdbgout.c: Revert unused header file reduction patch.
    
    git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@229650 138bc75d-0d04-0410-961f-82ee72b054a4

Comments

Andrew MacLeod Nov. 4, 2015, 3:51 p.m. UTC | #1
On 11/02/2015 12:28 AM, Jeff Law wrote:
>
> The header reduction didn't seem to handle the vms targets correctly. 
> This reverts that part of Andrew's patch which allows the alpha, 
> alpha64 and ia64 vms targets to build again.
>
> Installed on the trunk.  That covers all the fallout from standard 
> builds of config-list.mk that I'm aware of.
>
how weird.. really.  I ran all config-list.mk and saw no failures... 
huh.   maybe because I only ran c/c++ instead of 'all' to save time?  
I'm still surprised.

Andrew
Jeff Law Nov. 4, 2015, 10:01 p.m. UTC | #2
On 11/04/2015 08:51 AM, Andrew MacLeod wrote:
> On 11/02/2015 12:28 AM, Jeff Law wrote:
>>
>> The header reduction didn't seem to handle the vms targets correctly.
>> This reverts that part of Andrew's patch which allows the alpha,
>> alpha64 and ia64 vms targets to build again.
>>
>> Installed on the trunk.  That covers all the fallout from standard
>> builds of config-list.mk that I'm aware of.
>>
> how weird.. really.  I ran all config-list.mk and saw no failures...
> huh.   maybe because I only ran c/c++ instead of 'all' to save time? I'm
> still surprised.
It was a link failure with a reference from dwarf2out.o that should have 
been satisfied by vmsdbgout.o IIRC.

As why I saw it and not you.  I have no clue as I'd expect it to have 
failed linking cc1 or cc1plus.

Jeff
diff mbox

Patch

diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index 1ce0d36..d927d35 100644
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,5 +1,7 @@ 
 2015-11-01  Jeff Law <jeff@redhat.com>
 
+	* vmsdbgout.c: Revert unused header file reduction patch.
+
 	* config/mcore/mcore.c: Include regs.h.
 
 2015-10-31  Segher Boessenkool  <segher@kernel.crashing.org>
diff --git a/gcc/vmsdbgout.c b/gcc/vmsdbgout.c
index 26dd590..d41d4b2 100644
--- a/gcc/vmsdbgout.c
+++ b/gcc/vmsdbgout.c
@@ -22,6 +22,7 @@  along with GCC; see the file COPYING3.  If not see
 #include "config.h"
 #include "system.h"
 #include "coretypes.h"
+#include "tm.h"
 
 #ifdef VMS_DEBUGGING_INFO
 #include "alias.h"