sanchayanmaity/gem5
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

proto: Fix warnings for protoc v3

Browse source 
protoc v3 introduces a new syntax for proto files and warns when the
syntax is not explicitly stated.

protoc relies on the fact that undefined preprocessor symbols are
explanded to 0 but since we use -Wundef they end up generating
warnings.

Change-Id: If07abeb54e932469c8f2c4d38634a97fdae40f77
Reviewed-by: Andreas Hansson <andreas.hansson@arm.com>
Reviewed-by: Andreas Sandberg <andreas.sandberg@arm.com>
Signed-off-by: Jason Lowe-Power <jason@lowepower.com>
This commit is contained in:
Nikos Nikoleris 2017-01-27 15:07:20 -06:00
parent e4c57275d3
commit 227bdde922
4 changed files with 14 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
src/proto/SConscript
View file

@ -1,6 +1,6 @@
# -*- mode:python -*-
# Copyright (c) 2012 ARM Limited
# Copyright (c) 2012,2017 ARM Limited
# All rights reserved.
#
# The license below extends only to copyright in the software and shall
@ -45,3 +45,8 @@ if env['HAVE_PROTOBUF']:
ProtoBuf('packet.proto')
ProtoBuf('inst.proto')
Source('protoio.cc')
# protoc relies on the fact that undefined preprocessor symbols are
# explanded to 0 but since we use -Wundef they end up generating
# warnings.
env.Append(CCFLAGS='-DPROTOBUF_INLINE_NOT_IN_HEADERS=0')

3
src/proto/inst.proto
View file

@ -1,4 +1,4 @@
// Copyright (c) 2014 ARM Limited
// Copyright (c) 2014,2017 ARM Limited
// All rights reserved
//
// The license below extends only to copyright in the software and shall
@ -35,6 +35,7 @@
//
// Authors: Ali Saidi
syntax = "proto2";
// Put all the generated messages in a namespace
package ProtoMessage;

4
src/proto/inst_dep_record.proto
View file

@ -1,4 +1,4 @@
// Copyright (c) 2013 ARM Limited
// Copyright (c) 2013,2017 ARM Limited
// All rights reserved
//
// The license below extends only to copyright in the software and shall
@ -35,6 +35,8 @@
//
// Authors: Radhika Jagtap
syntax = "proto2";
// Put all the generated messages in a namespace
package ProtoMessage;

4
src/proto/packet.proto
View file

@ -1,4 +1,4 @@
// Copyright (c) 2012-2013 ARM Limited
// Copyright (c) 2012-2013,2017 ARM Limited
// All rights reserved
//
// The license below extends only to copyright in the software and shall
@ -35,6 +35,8 @@
//
// Authors: Andreas Hansson
syntax = "proto2";
// Put all the generated messages in a namespace
package ProtoMessage;