[PATCH] D85838: New TableGen Programmer's Reference document

Jay Foad via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 23 07:20:13 PDT 2022


foad added inline comments.


================
Comment at: llvm/docs/TableGen/ProgRef.rst:1153
+
+The ``foreach`` statement can also be used in a record :token:`Body`.
+
----------------
Paul-C-Anagnostopoulos wrote:
> foad wrote:
> > Is this true? This assertion did not appear in the previous documentation, and it doesn't seem to be borne out by the grammar productions: `BodyItem` only includes `let`, `defvar` and `assert` statements.
> > 
> > @Joe_Nash for awareness.
> It is not true. Feel free to get rid of that sentence. Good catch.
Thanks. 9293539064aead8d1822e181b66fd5590c062a1d.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D85838



More information about the llvm-commits mailing list