[mlir rebase] Add MLIR config and react to MLIR name changes

flang-compiler / f18

F18 is a front-end for Fortran intended to replace the existing front-end in the Flang compiler

230 stars 48 forks source link

[mlir rebase] Add MLIR config and react to MLIR name changes #1090

Closed sscalpone closed 4 years ago

sscalpone commented 4 years ago

Similar to #1085. Now use the MLIR package to set up paths for include files and libraries. Three MLIR names changed:

VectorOpsDialect to VectorDialect
AffineOpsDialect to AffineDialect
createVectorizePass to createSuperVectorizePass

Also, resolves the merge conflict in #1085.

Note, when building out of tree, supply MLIR_DIR to cmake:

cmake -DMLIR_DIR=/.../install/lib/cmake/mlir ...

tskeith commented 4 years ago

This doesn't build for me. I'm getting this:

CMake Error at CMakeLists.txt:70 (find_package):
  Could not find a package configuration file provided by "MLIR" with any of
  the following names:

    MLIRConfig.cmake
    mlir-config.cmake

I'm using the latest on the f18 branch from flang-compiler/f18-llvm-project. There is no lib/cmake/mlir in the install.

Can you update the build instructions?

sscalpone commented 4 years ago

Can you update the build instructions?

Jean is updating f18-llvm-project.

jeanPerier commented 4 years ago

I'm using the latest on the f18 branch from flang-compiler/f18-llvm-project. There is no lib/cmake/mlir in the install.

I have actually updated it a few hours ago already, can you check with git log --pretty=oneline -n 1 or similar that your f18-lvm-project f18 head is:

b3cc5dcef0f29f166fe33022858205a1243abd80 [MCInstPrinter] Add parameter `Address` to MCInstPrinter::printAliasInstr. NFC