[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