This is patch is to add PowerPC vector intrinsics for arithmetic operations of two vectors.
Details
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
flang/include/flang/Optimizer/Builder/IntrinsicCall.h | ||
---|---|---|
92 | typo: // -> /// | |
flang/lib/Optimizer/Dialect/FIROps.cpp | ||
1009 | Please use free function variants for isa (see https://reviews.llvm.org/D151556 for details). | |
flang/module/__fortran_ppc_intrinsics.f90 | ||
34 | Does Flang support unsigned? |
flang/module/__fortran_ppc_intrinsics.f90 | ||
---|---|---|
34 | Yes, but only for PPC vectors. |
flang/lib/Optimizer/Dialect/FIROps.cpp | ||
---|---|---|
1009 | Will add the check for the element types as well as size. |
Changes in this update:
- rebase
- fix the CI build breakage
- enhance checking in conversion between fir vector and mlir vector
flang/lib/Semantics/resolve-names.cpp | ||
---|---|---|
4855 | It would be more informative to be clear that the vector type is only available on PowerPC. |
flang/lib/Semantics/semantics.cpp | ||
---|---|---|
523 | This seems no longer needed as it is checking the target from targetCharacteristics. |
flang/lib/Semantics/semantics.cpp | ||
---|---|---|
526 | Actually, this checking is not needed also as the __fortran_ppc_types will always built. |
flang/lib/Semantics/semantics.cpp | ||
---|---|---|
526 | Yes. That module is always built. It does not need the check. I will remove the check. Thanks. |
typo: // -> ///