scons: Fix warnings issued by clang 3.2svn (XCode 4.6)
authorAndreas Hansson <andreas.hansson@arm.com>
Tue, 19 Feb 2013 10:56:08 +0000 (05:56 -0500)
committerAndreas Hansson <andreas.hansson@arm.com>
Tue, 19 Feb 2013 10:56:08 +0000 (05:56 -0500)
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.

23 files changed:
src/arch/alpha/linux/system.hh
src/arch/arm/isa/insts/neon.isa
src/arch/x86/interrupts.cc
src/arch/x86/isa/decoder/one_byte_opcodes.isa
src/base/statistics.hh
src/cpu/testers/traffic_gen/traffic_gen.hh
src/dev/Ethernet.py
src/dev/alpha/tsunami_io.hh
src/dev/i8254xGBe.cc
src/dev/i8254xGBe.hh
src/dev/x86/intdev.hh
src/mem/ruby/common/Consumer.hh
src/mem/ruby/filters/BlockBloomFilter.hh
src/mem/ruby/filters/BulkBloomFilter.hh
src/mem/ruby/filters/H3BloomFilter.hh
src/mem/ruby/filters/MultiBitSelBloomFilter.hh
src/mem/ruby/filters/MultiGrainBloomFilter.hh
src/mem/ruby/filters/NonCountingBloomFilter.hh
src/mem/ruby/network/garnet/fixed-pipeline/flitBuffer_d.hh
src/mem/ruby/network/garnet/flexible-pipeline/flitBuffer.hh
src/mem/ruby/system/DMASequencer.hh
src/mem/ruby/system/RubyPort.cc
src/mem/ruby/system/RubyPort.hh

index 345c17bb7d5d2ea07c0b01e04017d16b0b45aba3..4f030c99fddc6f32e3554437860d60f415036c87 100644 (file)
@@ -89,8 +89,11 @@ class LinuxAlphaSystem : public AlphaSystem
     /** Event to halt the simulator if the kernel calls panic()  */
     BreakPCEvent *kernelPanicEvent;
 
+#if 0
     /** Event to halt the simulator if the kernel calls die_if_kernel  */
     BreakPCEvent *kernelDieEvent;
+#endif
+
 #endif
 
     /**
index 3296b3b555949a3e02da07417b1eac3de84a877f..876bb3bb7fbf8ce544c02f1b169a0426d0e7347e 100644 (file)
@@ -2820,6 +2820,7 @@ let {{
             4, vcvts2fpCode, fromInt = True)
 
     vcvts2hCode = '''
+        destElem = 0;
         FPSCR fpscr = (FPSCR) FpscrExc;
         float srcFp1 = bitsToFp(srcElem1, (float)0.0);
         if (flushToZero(srcFp1))
@@ -2836,6 +2837,7 @@ let {{
     twoRegNarrowMiscInst("vcvt", "NVcvts2h", "SimdCvtOp", ("uint16_t",), vcvts2hCode)
 
     vcvth2sCode = '''
+        destElem = 0;
         FPSCR fpscr = (FPSCR) FpscrExc;
         VfpSavedState state = prepFpState(VfpRoundNearest);
         __asm__ __volatile__("" : "=m" (srcElem1), "=m" (destElem)
index c693e6bbdace00c5f0b3f795a2108e107731638c..9983d7305e2a8c81def27fa35de05fc80856a04c 100644 (file)
@@ -627,7 +627,7 @@ X86ISA::Interrupts::Interrupts(Params * p) :
     pendingStartup(false), startupVector(0),
     startedUp(false), pendingUnmaskableInt(false),
     pendingIPIs(0), cpu(NULL),
-    intSlavePort(name() + ".int_slave", this, this, latency)
+    intSlavePort(name() + ".int_slave", this, this)
 {
     pioSize = PageBytes;
     memset(regs, 0, sizeof(regs));
index 040f5d04f11c1c4c90011ab7d88c0bd38cbf3d6e..a6173831cb8e3dc09662baa9ffbf6c29bd5064a5 100644 (file)
                         // Really only the LSB matters, but the decoder
                         // will sign extend it, and there's no easy way to
                         // specify only checking the first byte.
-                        -0x80: SyscallInst::int80('xc->syscall(Rax)',
-                            IsSyscall, IsNonSpeculative, IsSerializeAfter);
+                        0xffffffffffffff80:
+                            SyscallInst::int80('xc->syscall(Rax)',
+                                 IsSyscall, IsNonSpeculative, IsSerializeAfter);
                     }
                 }
                 0x6: decode MODE_SUBMODE {
index 13347b7339c1118db8ad8cadbcce612f3754c20d..3c1a556234604e13c5caf8b55b7a96033f0a5688 100644 (file)
@@ -1327,8 +1327,6 @@ class DistStor
     Counter max_track;
     /** The number of entries in each bucket. */
     Counter bucket_size;
-    /** The number of buckets. Equal to (max-min)/bucket_size. */
-    size_type buckets;
 
     /** The smallest value sampled. */
     Counter min_val;
index 75db025e599026b3e720c3e62341ee44244c7544..fa08f44612dcc4a086946e0bed493738f7dce499 100644 (file)
@@ -548,13 +548,6 @@ class TrafficGen : public MemObject
              * state is complete.
              */
             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 */
@@ -582,8 +575,7 @@ class TrafficGen : public MemObject
       public:
 
         TrafficGenPort(const std::string& name, TrafficGen& _owner)
-            : QueuedMasterPort(name, &_owner, queue), queue(_owner, *this),
-              owner(_owner)
+            : QueuedMasterPort(name, &_owner, queue), queue(_owner, *this)
         { }
 
       protected:
@@ -594,9 +586,6 @@ class TrafficGen : public MemObject
 
         MasterPacketQueue queue;
 
-        // Owner of the port
-        TrafficGen& owner;
-
     };
 
     TrafficGenPort port;
index 191aad3a19b21e8d8d13d9cd333a2d8da653f4a7..0072b90faaa273071cb904e3712758f176e644ee 100644 (file)
@@ -78,8 +78,6 @@ class IGbE(EtherDevice):
     cxx_header = "dev/i8254xGBe.hh"
     hardware_address = Param.EthernetAddr(NextEthernetAddr,
         "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")
     tx_fifo_size = Param.MemorySize('384kB', "Size of the tx FIFO")
     rx_desc_cache_size = Param.Int(64,
index 7477fb12422f3e2da077d1d8aae41ec9010b7bc4..684295551ba1f14565c850613cb93756eec14ed8 100644 (file)
@@ -51,8 +51,6 @@
  */
 class TsunamiIO : public BasicPioDevice
 {
-  private:
-    struct tm tm;
 
   protected:
 
index 9a3ddaeb7b7f4ca01cb81ea8d1bbd34d3f738fdd..ee24b3922cdce51c5df1d5b87244f1175cb037fb 100644 (file)
@@ -58,7 +58,6 @@ using namespace Net;
 
 IGbE::IGbE(const Params *p)
     : EtherDevice(p), etherInt(NULL),  drainManager(NULL),
-      useFlowControl(p->use_flow_control),
       rxFifo(p->rx_fifo_size), txFifo(p->tx_fifo_size), rxTick(false),
       txTick(false), txFifoTick(false), rxDmaPacket(false), pktOffset(0),
       fetchDelay(p->fetch_delay), wbDelay(p->wb_delay), 
index b8099fb1c92a6abb90bc6d13c09f22b9c102fb5f..29571271733fe9c586dbcc5b6db9a63d25e68a40 100644 (file)
@@ -70,9 +70,6 @@ class IGbE : public EtherDevice
     // The drain event if we have one
     DrainManager *drainManager;
 
-    // cached parameters from params struct
-    bool useFlowControl;
-
     // packet fifos
     PacketFifo rxFifo;
     PacketFifo txFifo;
index a94ca47cc57acb9acf1ba3bf2de4cdcbe8d4dc0b..99c361bae734cff26172f166e3fefaf5e7320b45 100644 (file)
@@ -66,11 +66,11 @@ class IntDev
     class IntSlavePort : public MessageSlavePort
     {
         IntDev * device;
-        Tick latency;
+
       public:
         IntSlavePort(const std::string& _name, MemObject* _parent,
-                     IntDev* dev, Tick _latency) :
-            MessageSlavePort(_name, _parent), device(dev), latency(_latency)
+                     IntDev* dev) :
+            MessageSlavePort(_name, _parent), device(dev)
         {
         }
 
index 33c78d78092ff2ad3acf495d1eff8cc862ddf6b2..c1b4d70b1d1b5cdcf44d533f65612cded5923eaa 100644 (file)
@@ -44,7 +44,7 @@ class Consumer
 {
   public:
     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:
     Tick m_last_scheduled_wakeup;
     std::set<Tick> m_scheduled_wakeups;
-    Tick m_last_wakeup;
     ClockedObject *em;
 
     class ConsumerEvent : public Event
index ade599f4213402b813b0f237ba55b2f7beca2b40..e7958df62e4a006c18397d81c00c409074d865a9 100644 (file)
@@ -66,8 +66,6 @@ class BlockBloomFilter : public AbstractBloomFilter
     int m_filter_size;
     int m_filter_size_bits;
 
-    int m_count_bits;
-    int m_count;
 };
 
 #endif // __MEM_RUBY_FILTERS_BLOCKBLOOMFILTER_HH__
index 50b24732f08de01e34f01b5fd2edbb88586bda20..1a723c51d300ced6cf54c3c5e6621f09c72f6b7e 100644 (file)
@@ -71,8 +71,6 @@ class BulkBloomFilter : public AbstractBloomFilter
 
     int m_sector_bits;
 
-    int m_count_bits;
-    int m_count;
 };
 
 #endif // __MEM_RUBY_FILTERS_BULKBLOOMFILTER_HH__
index b040af42fb7a8743276f7ff582c560eaa3ce9868..c1b53c9b5c54919299913332910545db65773a91 100644 (file)
@@ -80,9 +80,6 @@ class H3BloomFilter : public AbstractBloomFilter
     int m_par_filter_size;
     int m_par_filter_size_bits;
 
-    int m_count_bits;
-    int m_count;
-
     int primes_list[6];// = {9323,11279,10247,30637,25717,43711};
     int mults_list[6]; //= {255,29,51,3,77,43};
     int adds_list[6]; //= {841,627,1555,241,7777,65391};
index 5263309a435413500219549156ac0be11bed48ab..f9a423648beba055a46bccf52b8d45f1f1f686fd 100644 (file)
@@ -82,9 +82,6 @@ class MultiBitSelBloomFilter : public AbstractBloomFilter
     int m_par_filter_size;
     int m_par_filter_size_bits;
 
-    int m_count_bits;
-    int m_count;
-
     bool isParallel;
 };
 
index dfe9c3e9864090d018d74bcf49854ab09aa7c749..d5a6e330c2fccf13a5954e078f8fdec8122835d4 100644 (file)
@@ -71,9 +71,6 @@ class MultiGrainBloomFilter : public AbstractBloomFilter
     std::vector<int> m_page_filter;
     int m_page_filter_size;
     int m_page_filter_size_bits;
-
-    int m_count_bits;
-    int m_count;
 };
 
 #endif // __MEM_RUBY_FILTERS_MULTIGRAINBLOOMFILTER_HH__
index 57274ef87e59efd8bac165c259e23d06f75d0b67..c940a0805169196793545f043c3199a8d339ffe2 100644 (file)
@@ -73,9 +73,6 @@ class NonCountingBloomFilter : public AbstractBloomFilter
     int m_filter_size;
     int m_offset;
     int m_filter_size_bits;
-
-    int m_count_bits;
-    int m_count;
 };
 
 #endif // __MEM_RUBY_FILTERS_NONCOUNTINGBLOOMFILTER_HH__
index 444cfae5cdc4097e67cd672b888c2a8d8d686992..4c1cae8425e0c7af26b5aaa66c28a25bb6b8ef76 100644 (file)
@@ -75,7 +75,7 @@ class flitBuffer_d
 
   private:
     std::vector<flit_d *> m_buffer;
-    int size, max_size;
+    int max_size;
 };
 
 inline std::ostream&
index 4b244a608df9bf35c506eed89e7286ddd2d2ee77..609c5a9b66c24f54cda71aafc0a03732463388d4 100644 (file)
@@ -58,7 +58,7 @@ class flitBuffer
 
   private:
     std::vector<flit *> m_buffer;
-    int size, max_size;
+    int max_size;
 };
 
 inline std::ostream&
index d8a6ef0598f0597556542de3daf9028c6bda8f5b..b3ba0be8c2e10a5cab05f21febc82e373aeb6052 100644 (file)
@@ -73,7 +73,6 @@ class DMASequencer : public RubyPort
     bool m_is_busy;
     uint64_t m_data_block_mask;
     DMARequest active_request;
-    int num_active_requests;
 };
 
 #endif // __MEM_RUBY_SYSTEM_DMASEQUENCER_HH__
index 5e9e8cdd479cfae20b86016cd081ece0013c38d3..4cfc1f252440916cbd7f1ccc97a75e21b00b03f6 100644 (file)
@@ -119,8 +119,7 @@ RubyPort::getSlavePort(const std::string &if_name, PortID idx)
 
 RubyPort::PioPort::PioPort(const std::string &_name,
                            RubyPort *_port)
-    : QueuedMasterPort(_name, _port, queue), queue(*_port, *this),
-      ruby_port(_port)
+    : QueuedMasterPort(_name, _port, queue), queue(*_port, *this)
 {
     DPRINTF(RubyPort, "creating master port on ruby sequencer %s\n", _name);
 }
index 3c61eb52225ae86614c3fb27aba57c0153d55caf..70f74d83b07ee018f86932a4f0369d214be468ab 100644 (file)
@@ -99,8 +99,6 @@ class RubyPort : public MemObject
 
         MasterPacketQueue queue;
 
-        RubyPort *ruby_port;
-
       public:
         PioPort(const std::string &_name, RubyPort *_port);