[Mlir-commits] [mlir] [mlir][LLVM] Add builders for llvm.intr.assume (PR #113317)

Sirui Mu llvmlistbot at llvm.org
Fri Oct 25 20:16:48 PDT 2024


================
@@ -87,6 +88,13 @@ class GEPArg : public PointerUnion<Value, GEPConstantIndex> {
 } // namespace LLVM
 } // namespace mlir
 
+namespace mlir {
+namespace LLVM {
+struct AssumeAlignTag {};
+struct AssumeSeparateStorageTag {};
----------------
Lancern wrote:

I tried this and it did not work because tblgen puts `extraClassDeclaration` at the end of the class declaration, and the builder declarations could not find them:

```cpp
class AssumeOp {
  static AssumeOp build(AlignTag);  // <-- builders generated
  struct AlignTag {};  // <-- extraClassDeclaration is put after builders thus builders could not find them
};
```

https://github.com/llvm/llvm-project/pull/113317


More information about the Mlir-commits mailing list