Skip to content

Commit 0e55364

Browse files
authored
Merge pull request #44984 from mbluj/CMSSW_14_1_X_omtf2constfix
[DBG] Fix undefined ref errors in L1TMuonOverlapPhase2
2 parents 9116e74 + 3f21c28 commit 0e55364

File tree

3 files changed

+31
-31
lines changed

3 files changed

+31
-31
lines changed

L1Trigger/L1TMuonOverlapPhase2/interface/LutNetworkFixedPointRegression2Outputs.h

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -63,9 +63,9 @@ namespace lutNN {
6363
LutLayer1;
6464
LutLayer1 lutLayer1;
6565

66-
static const unsigned int noHitCntShift = layer1_output_I; //FIXME should be layer1_output_I ???
66+
static constexpr unsigned int noHitCntShift = layer1_output_I; //FIXME should be layer1_output_I ???
6767

68-
static const int layer2_input_F = layer1_lut_F;
68+
static constexpr int layer2_input_F = layer1_lut_F;
6969

7070
typedef LutNeuronLayerFixedPoint<layer2_input_I,
7171
layer2_input_F,
@@ -77,7 +77,7 @@ namespace lutNN {
7777
LutLayer2;
7878
LutLayer2 lutLayer2;
7979

80-
static const int layer3_input_F = layer2_lut_F;
80+
static constexpr int layer3_input_F = layer2_lut_F;
8181

8282
typedef LutNeuronLayerFixedPoint<layer3_input_I,
8383
layer3_input_F,

L1Trigger/L1TMuonOverlapPhase2/interface/LutNeuronLayerFixedPoint.h

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -41,18 +41,18 @@ namespace lutNN {
4141
template <int input_I, int input_F, size_t inputSize, int lut_I, int lut_F, int neurons, int output_I>
4242
class LutNeuronLayerFixedPoint {
4343
public:
44-
static const int input_W = input_I + input_F;
45-
static const int lut_W = lut_I + lut_F;
44+
static constexpr int input_W = input_I + input_F;
45+
static constexpr int lut_W = lut_I + lut_F;
4646

4747
//the lut out values sum
4848
//static const int lutOutSum_I = lut_I + ceil(log2(inputSize)); //MB: ceil(log2(inputSize)) is not constexpr which makes issue for code-checks
49-
static const int lutOutSum_I = lut_I + ceillog2(inputSize);
50-
static const int lutOutSum_W = lutOutSum_I + lut_F;
49+
static constexpr int lutOutSum_I = lut_I + ceillog2(inputSize);
50+
static constexpr int lutOutSum_W = lutOutSum_I + lut_F;
5151

52-
static const int output_W = output_I + lut_F;
52+
static constexpr int output_W = output_I + lut_F;
5353

5454
//static_assert( (1<<input_I) <= lutSize);
55-
static const size_t lutSize = 1 << input_I;
55+
static constexpr size_t lutSize = 1 << input_I;
5656

5757
typedef std::array<ap_ufixed<input_W, input_I, AP_TRN, AP_SAT>, inputSize> inputArrayType;
5858

L1Trigger/L1TMuonOverlapPhase2/src/PtAssignmentNNRegression.cc

Lines changed: 22 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -21,35 +21,35 @@
2121
#include <fstream>
2222

2323
namespace lutNN {
24-
static const int input_I = 10;
25-
static const int input_F = 4;
26-
static const std::size_t networkInputSize = 18;
24+
static constexpr int input_I = 10;
25+
static constexpr int input_F = 4;
26+
static constexpr std::size_t networkInputSize = 18;
2727

28-
static const int layer1_neurons = 16;
29-
static const int layer1_lut_I = 3;
30-
static const int layer1_lut_F = 13;
28+
static constexpr int layer1_neurons = 16;
29+
static constexpr int layer1_lut_I = 3;
30+
static constexpr int layer1_lut_F = 13;
3131

32-
static const int layer1_output_I = 4;
32+
static constexpr int layer1_output_I = 4;
3333
//4 bits are for the count of the noHit layers which goes to the input of the layer2
34-
static const int layer2_input_I = 8;
34+
static constexpr int layer2_input_I = 8;
3535

36-
static const int layer2_neurons = 9;
37-
static const int layer2_lut_I = 5;
38-
static const int layer2_lut_F = 11;
36+
static constexpr int layer2_neurons = 9;
37+
static constexpr int layer2_lut_I = 5;
38+
static constexpr int layer2_lut_F = 11;
3939

40-
static const int layer3_input_I = 5;
40+
static constexpr int layer3_input_I = 5;
4141

42-
static const int layer3_0_inputCnt = 8;
43-
static const int layer3_0_lut_I = 5;
44-
static const int layer3_0_lut_F = 11;
45-
static const int output0_I = 8;
46-
static const int output0_F = 2;
42+
static constexpr int layer3_0_inputCnt = 8;
43+
static constexpr int layer3_0_lut_I = 5;
44+
static constexpr int layer3_0_lut_F = 11;
45+
static constexpr int output0_I = 8;
46+
static constexpr int output0_F = 2;
4747

48-
static const int layer3_1_inputCnt = 1;
49-
static const int layer3_1_lut_I = 4;
50-
static const int layer3_1_lut_F = 11;
51-
static const int output1_I = 8;
52-
static const int output1_F = 0; //Does not matter in principle - it is not used
48+
static constexpr int layer3_1_inputCnt = 1;
49+
static constexpr int layer3_1_lut_I = 4;
50+
static constexpr int layer3_1_lut_F = 11;
51+
static constexpr int output1_I = 8;
52+
static constexpr int output1_F = 0; //Does not matter in principle - it is not used
5353

5454
typedef LutNetworkFixedPointRegression2Outputs<input_I,
5555
input_F,

0 commit comments

Comments
 (0)