[PATCH] D84201: [LangRef] Add integer wrapping operand bundles.

Johannes Doerfert via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 20 22:34:27 PDT 2020


jdoerfert added inline comments.


================
Comment at: llvm/docs/LangRef.rst:2306
+
+
 .. _moduleasm:
----------------
[Drive By]

Should we add a reference to nuw/nsw specification (if we have one)?

I'm not sure the the description is explicit enough. `if
unsigned and/or signed overflow, respectively, occurs`, where/when?


Nit: I would say the type is spelled `i1` but I don't care much.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D84201





More information about the llvm-commits mailing list