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

Sirui Mu llvmlistbot at llvm.org
Thu Oct 24 06:08:18 PDT 2024


Lancern wrote:

I have updated the patch and kept only the new builders for `llvm.intr.assume`. Specifically, the new patch adds 4 builders:

```
OpBuilder<(ins "Value":$cond,
               "ArrayRef<llvm::OperandBundleDefT<Value>>":$opBundles)>

OpBuilder<(ins "Value":$cond, "llvm::StringRef":$tag, "ValueRange":$args)>

OpBuilder<(ins "Value":$cond, "AssumeAlignTag":$tag, "Value":$ptr,
               "Value":$align)>

OpBuilder<(ins "Value":$cond, "AssumeSeparateStorageTag":$tag,
               "Value":$ptr1, "Value":$ptr2)>
```

The first two are general and the last two are for specific tags. Currently I only add builders for `align` and `separate_storage` since this is the only two tags I'm interested in. We could add more builders if people are interested in other tags.

As for tests, since we don't have any tests yet for operation builders, I assume it's safe to ignore them for now.

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


More information about the Mlir-commits mailing list