[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