Merge pull request #41523 from smuzaffar/l1-llvm16-warn3

[L1] [CLANG] Fix warnings reported by llvm16 in CLANG IBs
cms-sw · May 6, 2023 · a52f2d1 · a52f2d1
2 parents b0972fd + 62baf85
commit a52f2d1
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 7 deletions.
diff --git a/DataFormats/L1GlobalTrigger/src/L1GlobalTriggerRecord.cc b/DataFormats/L1GlobalTrigger/src/L1GlobalTriggerRecord.cc
@@ -138,7 +138,6 @@ void L1GlobalTriggerRecord::printGtDecision(std::ostream& myCout) const {
   int sizeW64 = 64;  // 64 bits words
 
   int iBit = 0;
-  int jBit = m_gtDecisionWord.size();
   int nrDecWord = m_gtDecisionWord.size() / sizeW64;
 
   std::ostringstream stream64;
@@ -162,7 +161,6 @@ void L1GlobalTriggerRecord::printGtDecision(std::ostream& myCout) const {
     }
 
     iBit++;
-    jBit--;
   }
 
   int iWord = 0;

diff --git a/DataFormats/L1GlobalTrigger/src/L1GtFdlWord.cc b/DataFormats/L1GlobalTrigger/src/L1GtFdlWord.cc
@@ -288,7 +288,6 @@ void L1GtFdlWord::printGtDecisionWord(std::ostream& myCout) const {
   int sizeW64 = 64;  // 64 bits words
 
   int iBit = 0;
-  int jBit = m_gtDecisionWord.size();
   int nrDecWord = m_gtDecisionWord.size() / sizeW64;
 
   std::ostringstream stream64;
@@ -312,7 +311,6 @@ void L1GtFdlWord::printGtDecisionWord(std::ostream& myCout) const {
     }
 
     iBit++;
-    jBit--;
   }
 
   int iWord = 0;

diff --git a/DataFormats/L1Trigger/src/L1DataEmulRecord.cc b/DataFormats/L1Trigger/src/L1DataEmulRecord.cc
@@ -11,9 +11,9 @@ L1DataEmulRecord::L1DataEmulRecord() : deAgree(false), deGlt() {
 }
 
 L1DataEmulRecord::L1DataEmulRecord(bool evt_match,
-                                   bool sys_comp[],
-                                   bool sys_match[],
-                                   int nCand[][2],
+                                   bool sys_comp[DEnsys],
+                                   bool sys_match[DEnsys],
+                                   int nCand[DEnsys][2],
                                    const L1DEDigiCollection& coll,
                                    const GltDEDigi& glt)
     : deAgree(evt_match), deGlt(glt) {