scons: Fix warnings issued by clang 3.2svn (XCode 4.6)

This patch fixes the warnings that clang3.2svn emit due to the "-Wall"
flag. There is one case of an uninitialised value in the ARM neon ISA
description, and then a whole range of unused private fields that are
pruned.
This commit is contained in:
Andreas Hansson 2013-02-19 05:56:08 -05:00
parent 08a5fd328b
commit a62afd094b
23 changed files with 17 additions and 53 deletions

View file

@ -89,8 +89,11 @@ class LinuxAlphaSystem : public AlphaSystem
/** Event to halt the simulator if the kernel calls panic() */ /** Event to halt the simulator if the kernel calls panic() */
BreakPCEvent *kernelPanicEvent; BreakPCEvent *kernelPanicEvent;
#if 0
/** Event to halt the simulator if the kernel calls die_if_kernel */ /** Event to halt the simulator if the kernel calls die_if_kernel */
BreakPCEvent *kernelDieEvent; BreakPCEvent *kernelDieEvent;
#endif
#endif #endif
/** /**

View file

@ -2820,6 +2820,7 @@ let {{
4, vcvts2fpCode, fromInt = True) 4, vcvts2fpCode, fromInt = True)
vcvts2hCode = ''' vcvts2hCode = '''
destElem = 0;
FPSCR fpscr = (FPSCR) FpscrExc; FPSCR fpscr = (FPSCR) FpscrExc;
float srcFp1 = bitsToFp(srcElem1, (float)0.0); float srcFp1 = bitsToFp(srcElem1, (float)0.0);
if (flushToZero(srcFp1)) if (flushToZero(srcFp1))
@ -2836,6 +2837,7 @@ let {{
twoRegNarrowMiscInst("vcvt", "NVcvts2h", "SimdCvtOp", ("uint16_t",), vcvts2hCode) twoRegNarrowMiscInst("vcvt", "NVcvts2h", "SimdCvtOp", ("uint16_t",), vcvts2hCode)
vcvth2sCode = ''' vcvth2sCode = '''
destElem = 0;
FPSCR fpscr = (FPSCR) FpscrExc; FPSCR fpscr = (FPSCR) FpscrExc;
VfpSavedState state = prepFpState(VfpRoundNearest); VfpSavedState state = prepFpState(VfpRoundNearest);
__asm__ __volatile__("" : "=m" (srcElem1), "=m" (destElem) __asm__ __volatile__("" : "=m" (srcElem1), "=m" (destElem)

View file

@ -627,7 +627,7 @@ X86ISA::Interrupts::Interrupts(Params * p) :
pendingStartup(false), startupVector(0), pendingStartup(false), startupVector(0),
startedUp(false), pendingUnmaskableInt(false), startedUp(false), pendingUnmaskableInt(false),
pendingIPIs(0), cpu(NULL), pendingIPIs(0), cpu(NULL),
intSlavePort(name() + ".int_slave", this, this, latency) intSlavePort(name() + ".int_slave", this, this)
{ {
pioSize = PageBytes; pioSize = PageBytes;
memset(regs, 0, sizeof(regs)); memset(regs, 0, sizeof(regs));

View file

@ -399,8 +399,9 @@
// Really only the LSB matters, but the decoder // Really only the LSB matters, but the decoder
// will sign extend it, and there's no easy way to // will sign extend it, and there's no easy way to
// specify only checking the first byte. // specify only checking the first byte.
-0x80: SyscallInst::int80('xc->syscall(Rax)', 0xffffffffffffff80:
IsSyscall, IsNonSpeculative, IsSerializeAfter); SyscallInst::int80('xc->syscall(Rax)',
IsSyscall, IsNonSpeculative, IsSerializeAfter);
} }
} }
0x6: decode MODE_SUBMODE { 0x6: decode MODE_SUBMODE {

View file

@ -1327,8 +1327,6 @@ class DistStor
Counter max_track; Counter max_track;
/** The number of entries in each bucket. */ /** The number of entries in each bucket. */
Counter bucket_size; Counter bucket_size;
/** The number of buckets. Equal to (max-min)/bucket_size. */
size_type buckets;
/** The smallest value sampled. */ /** The smallest value sampled. */
Counter min_val; Counter min_val;

View file

@ -548,13 +548,6 @@ class TrafficGen : public MemObject
* state is complete. * state is complete.
*/ */
bool traceComplete; bool traceComplete;
/**
* Used to store the Tick when the next generate should
* occur. It is to remove a transaction as soon as we
* enter the state.
*/
Tick oldEmitTime;
}; };
/** Pointer to owner of request handler */ /** Pointer to owner of request handler */
@ -582,8 +575,7 @@ class TrafficGen : public MemObject
public: public:
TrafficGenPort(const std::string& name, TrafficGen& _owner) TrafficGenPort(const std::string& name, TrafficGen& _owner)
: QueuedMasterPort(name, &_owner, queue), queue(_owner, *this), : QueuedMasterPort(name, &_owner, queue), queue(_owner, *this)
owner(_owner)
{ } { }
protected: protected:
@ -594,9 +586,6 @@ class TrafficGen : public MemObject
MasterPacketQueue queue; MasterPacketQueue queue;
// Owner of the port
TrafficGen& owner;
}; };
TrafficGenPort port; TrafficGenPort port;

View file

@ -78,8 +78,6 @@ class IGbE(EtherDevice):
cxx_header = "dev/i8254xGBe.hh" cxx_header = "dev/i8254xGBe.hh"
hardware_address = Param.EthernetAddr(NextEthernetAddr, hardware_address = Param.EthernetAddr(NextEthernetAddr,
"Ethernet Hardware Address") "Ethernet Hardware Address")
use_flow_control = Param.Bool(False,
"Should we use xon/xoff flow contorl (UNIMPLEMENTD)")
rx_fifo_size = Param.MemorySize('384kB', "Size of the rx FIFO") rx_fifo_size = Param.MemorySize('384kB', "Size of the rx FIFO")
tx_fifo_size = Param.MemorySize('384kB', "Size of the tx FIFO") tx_fifo_size = Param.MemorySize('384kB', "Size of the tx FIFO")
rx_desc_cache_size = Param.Int(64, rx_desc_cache_size = Param.Int(64,

View file

@ -51,8 +51,6 @@
*/ */
class TsunamiIO : public BasicPioDevice class TsunamiIO : public BasicPioDevice
{ {
private:
struct tm tm;
protected: protected:

View file

@ -58,7 +58,6 @@ using namespace Net;
IGbE::IGbE(const Params *p) IGbE::IGbE(const Params *p)
: EtherDevice(p), etherInt(NULL), drainManager(NULL), : EtherDevice(p), etherInt(NULL), drainManager(NULL),
useFlowControl(p->use_flow_control),
rxFifo(p->rx_fifo_size), txFifo(p->tx_fifo_size), rxTick(false), rxFifo(p->rx_fifo_size), txFifo(p->tx_fifo_size), rxTick(false),
txTick(false), txFifoTick(false), rxDmaPacket(false), pktOffset(0), txTick(false), txFifoTick(false), rxDmaPacket(false), pktOffset(0),
fetchDelay(p->fetch_delay), wbDelay(p->wb_delay), fetchDelay(p->fetch_delay), wbDelay(p->wb_delay),

View file

@ -70,9 +70,6 @@ class IGbE : public EtherDevice
// The drain event if we have one // The drain event if we have one
DrainManager *drainManager; DrainManager *drainManager;
// cached parameters from params struct
bool useFlowControl;
// packet fifos // packet fifos
PacketFifo rxFifo; PacketFifo rxFifo;
PacketFifo txFifo; PacketFifo txFifo;

View file

@ -66,11 +66,11 @@ class IntDev
class IntSlavePort : public MessageSlavePort class IntSlavePort : public MessageSlavePort
{ {
IntDev * device; IntDev * device;
Tick latency;
public: public:
IntSlavePort(const std::string& _name, MemObject* _parent, IntSlavePort(const std::string& _name, MemObject* _parent,
IntDev* dev, Tick _latency) : IntDev* dev) :
MessageSlavePort(_name, _parent), device(dev), latency(_latency) MessageSlavePort(_name, _parent), device(dev)
{ {
} }

View file

@ -44,7 +44,7 @@ class Consumer
{ {
public: public:
Consumer(ClockedObject *_em) Consumer(ClockedObject *_em)
: m_last_scheduled_wakeup(0), m_last_wakeup(0), em(_em) : m_last_scheduled_wakeup(0), em(_em)
{ {
} }
@ -93,7 +93,6 @@ class Consumer
private: private:
Tick m_last_scheduled_wakeup; Tick m_last_scheduled_wakeup;
std::set<Tick> m_scheduled_wakeups; std::set<Tick> m_scheduled_wakeups;
Tick m_last_wakeup;
ClockedObject *em; ClockedObject *em;
class ConsumerEvent : public Event class ConsumerEvent : public Event

View file

@ -66,8 +66,6 @@ class BlockBloomFilter : public AbstractBloomFilter
int m_filter_size; int m_filter_size;
int m_filter_size_bits; int m_filter_size_bits;
int m_count_bits;
int m_count;
}; };
#endif // __MEM_RUBY_FILTERS_BLOCKBLOOMFILTER_HH__ #endif // __MEM_RUBY_FILTERS_BLOCKBLOOMFILTER_HH__

View file

@ -71,8 +71,6 @@ class BulkBloomFilter : public AbstractBloomFilter
int m_sector_bits; int m_sector_bits;
int m_count_bits;
int m_count;
}; };
#endif // __MEM_RUBY_FILTERS_BULKBLOOMFILTER_HH__ #endif // __MEM_RUBY_FILTERS_BULKBLOOMFILTER_HH__

View file

@ -80,9 +80,6 @@ class H3BloomFilter : public AbstractBloomFilter
int m_par_filter_size; int m_par_filter_size;
int m_par_filter_size_bits; int m_par_filter_size_bits;
int m_count_bits;
int m_count;
int primes_list[6];// = {9323,11279,10247,30637,25717,43711}; int primes_list[6];// = {9323,11279,10247,30637,25717,43711};
int mults_list[6]; //= {255,29,51,3,77,43}; int mults_list[6]; //= {255,29,51,3,77,43};
int adds_list[6]; //= {841,627,1555,241,7777,65391}; int adds_list[6]; //= {841,627,1555,241,7777,65391};

View file

@ -82,9 +82,6 @@ class MultiBitSelBloomFilter : public AbstractBloomFilter
int m_par_filter_size; int m_par_filter_size;
int m_par_filter_size_bits; int m_par_filter_size_bits;
int m_count_bits;
int m_count;
bool isParallel; bool isParallel;
}; };

View file

@ -71,9 +71,6 @@ class MultiGrainBloomFilter : public AbstractBloomFilter
std::vector<int> m_page_filter; std::vector<int> m_page_filter;
int m_page_filter_size; int m_page_filter_size;
int m_page_filter_size_bits; int m_page_filter_size_bits;
int m_count_bits;
int m_count;
}; };
#endif // __MEM_RUBY_FILTERS_MULTIGRAINBLOOMFILTER_HH__ #endif // __MEM_RUBY_FILTERS_MULTIGRAINBLOOMFILTER_HH__

View file

@ -73,9 +73,6 @@ class NonCountingBloomFilter : public AbstractBloomFilter
int m_filter_size; int m_filter_size;
int m_offset; int m_offset;
int m_filter_size_bits; int m_filter_size_bits;
int m_count_bits;
int m_count;
}; };
#endif // __MEM_RUBY_FILTERS_NONCOUNTINGBLOOMFILTER_HH__ #endif // __MEM_RUBY_FILTERS_NONCOUNTINGBLOOMFILTER_HH__

View file

@ -75,7 +75,7 @@ class flitBuffer_d
private: private:
std::vector<flit_d *> m_buffer; std::vector<flit_d *> m_buffer;
int size, max_size; int max_size;
}; };
inline std::ostream& inline std::ostream&

View file

@ -58,7 +58,7 @@ class flitBuffer
private: private:
std::vector<flit *> m_buffer; std::vector<flit *> m_buffer;
int size, max_size; int max_size;
}; };
inline std::ostream& inline std::ostream&

View file

@ -73,7 +73,6 @@ class DMASequencer : public RubyPort
bool m_is_busy; bool m_is_busy;
uint64_t m_data_block_mask; uint64_t m_data_block_mask;
DMARequest active_request; DMARequest active_request;
int num_active_requests;
}; };
#endif // __MEM_RUBY_SYSTEM_DMASEQUENCER_HH__ #endif // __MEM_RUBY_SYSTEM_DMASEQUENCER_HH__

View file

@ -119,8 +119,7 @@ RubyPort::getSlavePort(const std::string &if_name, PortID idx)
RubyPort::PioPort::PioPort(const std::string &_name, RubyPort::PioPort::PioPort(const std::string &_name,
RubyPort *_port) RubyPort *_port)
: QueuedMasterPort(_name, _port, queue), queue(*_port, *this), : QueuedMasterPort(_name, _port, queue), queue(*_port, *this)
ruby_port(_port)
{ {
DPRINTF(RubyPort, "creating master port on ruby sequencer %s\n", _name); DPRINTF(RubyPort, "creating master port on ruby sequencer %s\n", _name);
} }

View file

@ -99,8 +99,6 @@ class RubyPort : public MemObject
MasterPacketQueue queue; MasterPacketQueue queue;
RubyPort *ruby_port;
public: public:
PioPort(const std::string &_name, RubyPort *_port); PioPort(const std::string &_name, RubyPort *_port);