- User Since
- Jun 28 2018, 2:24 AM (133 w, 5 d)
https://github.com/llvm/llvm-project/blob/main/mlir/include/mlir/Dialect/LLVMIR/NVVMOps.td as an example. We also have support in mlir-tblgen to generate the boilerplate for ops given LLVM IR intrinsic definition file, example of usage: https://github.com/llvm/llvm-project/blob/main/mlir/test/mlir-tblgen/llvm-intrinsics.td
Intermediate coro/runtime operations will allow to run transformations on a higher level IR and do not try to match IR based on the LLVM::CallOp properties.
Although async.coro is very close to LLVM coroutines, it is not exactly the same API, instead it is optimized for usability in async lowering, and misses a lot of details that are present in @llvm.coro intrinsic.
We generally use code review for everything except build unbreaks, reverts and minor syntactic changes such as typos.
Not sure about this one... I can understand the rationale, but this starts to diverge from C++/ODS, which already has suboptimal documentation and I'm apprehensive of the complexity it might add. If we agree we want to put result types after operands, we should be doing so consistently and that could be a problem for generating builders automatically if we decide we want that.
Hmm, this change actually makes Python API align on C++ API. I don't remember if there was a good reason to swap the order in C.
Matches how all of the other shaped types are declared.
Fri, Jan 15
Thu, Jan 14
We can't test for this because the Euclidean GCD algorithm is still correct and terminates with machine division (about efficiency, I am not sure).
Please consider avoiding "advanced" markdown features in the commit messages, a lot of people look at them in the terminal and it may be hard to parse
Wed, Jan 13
Tue, Jan 12
I took your name and email from Github commits. Please consider sending patches with Arcanist to preserve author (and base revision) info.
Looks like a duplicate of D94496
Mon, Jan 11
Is it always unnecessary or is there a possibility that it still might be (e.g., async pointer)?
Fri, Jan 8
It looks like something went wrong when updating the second revision and the diff now only contains the comment change. Note that if you use arc diff on a branch of multiple commits, you must specify the commit that the branch is based on rather than HEAD^, which is merely the previous-to-head commit.
Thu, Jan 7
FYI, if you send commits with arcanist, the committer credentials are preserved.
Wed, Jan 6
Thanks for iterating on this!
Tue, Jan 5