Skip to content

Commit 532d484

Browse files
authored
[mlir][Quasipolynomials] Fixed type issues in GeneratorFuunction.h (#76413)
Fixed two issues: A SmallVector size that caused size-differences issue (8 vs. 12). Thus removed this size restriction. Also a constant parameter was causing an issue in a function not marked constant.
1 parent aacff34 commit 532d484

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

mlir/lib/Analysis/Presburger/GeneratingFunction.h

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,7 @@ using Point = SmallVector<Fraction>;
4949
// g_{ij} \in Q^n are vectors.
5050
class GeneratingFunction {
5151
public:
52-
GeneratingFunction(unsigned numParam, SmallVector<int, 8> signs,
52+
GeneratingFunction(unsigned numParam, SmallVector<int> signs,
5353
std::vector<ParamPoint> nums,
5454
std::vector<std::vector<Point>> dens)
5555
: numParam(numParam), signs(signs), numerators(nums), denominators(dens) {
@@ -67,7 +67,7 @@ class GeneratingFunction {
6767

6868
std::vector<std::vector<Point>> getDenominators() { return denominators; }
6969

70-
GeneratingFunction operator+(const GeneratingFunction &gf) const {
70+
GeneratingFunction operator+(GeneratingFunction &gf) const {
7171
assert(numParam == gf.getNumParams() &&
7272
"two generating functions with different numbers of parameters "
7373
"cannot be added!");
@@ -81,7 +81,7 @@ class GeneratingFunction {
8181
std::vector<std::vector<Point>> sumDenominators = denominators;
8282
sumDenominators.insert(sumDenominators.end(), gf.denominators.begin(),
8383
gf.denominators.end());
84-
return GeneratingFunction(sumSigns, sumNumerators, sumDenominators);
84+
return GeneratingFunction(0, sumSigns, sumNumerators, sumDenominators);
8585
}
8686

8787
llvm::raw_ostream &print(llvm::raw_ostream &os) const {
@@ -121,12 +121,12 @@ class GeneratingFunction {
121121

122122
private:
123123
unsigned numParam;
124-
SmallVector<int, 8> signs;
124+
SmallVector<int> signs;
125125
std::vector<ParamPoint> numerators;
126126
std::vector<std::vector<Point>> denominators;
127127
};
128128

129129
} // namespace presburger
130130
} // namespace mlir
131131

132-
#endif // MLIR_ANALYSIS_PRESBURGER_GENERATINGFUNCTION_H
132+
#endif // MLIR_ANALYSIS_PRESBURGER_GENERATINGFUNCTION_H

0 commit comments

Comments
 (0)