Skip to content

Commit

Permalink
Bump LLVM commit to f66cd9e9556a53142a26a5c21a72e21f1579217c and fix …
Browse files Browse the repository at this point in the history
…related compile issues (#2521)

Signed-off-by: Ferdinand Lemaire <[email protected]>
Co-authored-by: Philip Lassen <[email protected]>
  • Loading branch information
flemairen6 and philass authored Sep 21, 2023
1 parent ee11bc5 commit 6b6ee6b
Show file tree
Hide file tree
Showing 6 changed files with 66 additions and 66 deletions.
2 changes: 1 addition & 1 deletion docs/BuildOnLinuxOSX.md
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ Firstly, install MLIR (as a part of LLVM-Project):
``` bash
git clone -n https://github.com/llvm/llvm-project.git
# Check out a specific branch that is known to work with ONNX-MLIR.
cd llvm-project && git checkout 4acc3ffbb0af5631bc7916aeff3570f448899647 && cd ..
cd llvm-project && git checkout f66cd9e9556a53142a26a5c21a72e21f1579217c && cd ..
```

[same-as-file]: <> (utils/build-mlir.sh)
Expand Down
2 changes: 1 addition & 1 deletion docs/BuildOnWindows.md
Original file line number Diff line number Diff line change
Expand Up @@ -52,7 +52,7 @@ Install MLIR (as a part of LLVM-Project):
```shell
git clone -n https://github.com/llvm/llvm-project.git
# Check out a specific branch that is known to work with ONNX-MLIR.
cd llvm-project && git checkout 4acc3ffbb0af5631bc7916aeff3570f448899647 && cd ..
cd llvm-project && git checkout f66cd9e9556a53142a26a5c21a72e21f1579217c && cd ..
```

[same-as-file]: <> (utils/build-mlir.cmd)
Expand Down
2 changes: 1 addition & 1 deletion src/Conversion/KrnlToLLVM/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -35,6 +35,6 @@ add_onnx_mlir_library(OMKrnlToLLVM
MLIRReconcileUnrealizedCasts
MLIRSCFToControlFlow
MLIRShapeToStandard
MLIRVectorToLLVM
MLIRVectorToLLVMPass
onnx
)
4 changes: 2 additions & 2 deletions src/Dialect/Mlir/DialectBuilder.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -278,7 +278,7 @@ Value MathBuilder::floor(Value val) const {
Value MathBuilder::min(Value lhs, Value rhs) const {
assert(lhs.getType() == rhs.getType() && "expected same type");
if (isFloatWithVector(lhs.getType()))
return b().create<arith::MinFOp>(loc(), lhs, rhs);
return b().create<arith::MinNumFOp>(loc(), lhs, rhs);
if (isUnsignedIntegerWithVector(lhs.getType()))
return b().create<arith::MinUIOp>(loc(), lhs, rhs);
if (isIntegerWithVector(lhs.getType()))
Expand All @@ -289,7 +289,7 @@ Value MathBuilder::min(Value lhs, Value rhs) const {
Value MathBuilder::max(Value lhs, Value rhs) const {
assert(lhs.getType() == rhs.getType() && "expected same type");
if (isFloatWithVector(lhs.getType()))
return b().create<arith::MaxFOp>(loc(), lhs, rhs);
return b().create<arith::MaxNumFOp>(loc(), lhs, rhs);
if (isUnsignedIntegerWithVector(lhs.getType()))
return b().create<arith::MaxUIOp>(loc(), lhs, rhs);
if (isIntegerWithVector(lhs.getType()))
Expand Down
Loading

0 comments on commit 6b6ee6b

Please sign in to comment.