[PATCH] D85603: IR: Add convergence control operand bundle and intrinsics

[Nicolai Hähnle via Phabricator via llvm-commits]

nhaehnle updated this revision to Diff 284735.
nhaehnle marked 11 inline comments as done.
nhaehnle added a comment.
Herald added a subscriber: jfb.

With this change, I've edited the documents in a way where I hope all
comments have been addressed.

- Augmented the initial motivating examples with version that show how the control intrinsics are added
- Added an additional motivating example to illustrate what the "anchor" is for
- Changed the order in which control intrinsics are described, to hopefully make it even more obvious that the "less surprising" one is the entry intrinsic
- Add informational notes to the "Dynamic Instances and Convergence Tokens" section, and clean up some minor things in the Formal Rules
- A bunch of other random changes all over the place


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D85603/new/

https://reviews.llvm.org/D85603

Files:
  llvm/docs/ConvergentOperations.rst
  llvm/docs/LangRef.rst
  llvm/docs/Reference.rst
  llvm/include/llvm/IR/Intrinsics.td
  llvm/include/llvm/IR/LLVMContext.h
  llvm/lib/IR/LLVMContext.cpp
  llvm/test/Bitcode/operand-bundles-bc-analyzer.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D85603.284735.patch
Type: text/x-patch
Size: 44345 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200811/fc60d87b/attachment.bin>