[Mlir-commits] [mlir] [mlir][LLVM] Add builders for llvm.intr.assume (PR #113317)
Tobias Gysi
llvmlistbot at llvm.org
Thu Oct 24 09:07:29 PDT 2024
================
@@ -87,6 +88,13 @@ class GEPArg : public PointerUnion<Value, GEPConstantIndex> {
} // namespace LLVM
} // namespace mlir
+namespace mlir {
+namespace LLVM {
+struct AssumeAlignTag {};
+struct AssumeSeparateStorageTag {};
----------------
gysit wrote:
Can you try to add them in an extraClassDeclaration in the tablegen definition of the assume intrinsic?
```
let extraClassDeclaration = [{
struct AssumeAlignTag {};
struct AssumeSeparateStorageTag {};
}];
```
Afterwards they should be accessible as AssumeOp::AlignTag / AssumeOp::SeparateStorageTag.
I think that may work and would clearly link them with the op.
https://github.com/llvm/llvm-project/pull/113317
More information about the Mlir-commits
mailing list