[PATCH] D72693: [TableGen] Update editor modes for new keywords.

Simon Tatham via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 14 05:16:49 PST 2020


simon_tatham marked an inline comment as done.
simon_tatham added inline comments.


================
Comment at: llvm/utils/kate/llvm-tablegen.xml:11
+      <item> else </item>
+      <item> defvar </item>
     </list>
----------------
lebedev.ri wrote:
> simon_tatham wrote:
> > lebedev.ri wrote:
> > > I think `defvar` should go next to `def`
> > That would mean putting it in the 'class-like' section, which I thought probably wasn't right, on the grounds that `defvar` does not open a block with further declarations inside it, whereas `def` does, in common with the other 'class-like' keywords.
> > 
> > I admit I'm not a Kate user, though, so I don't know what difference (if any) that would actually make during editing.
> On a second thought, `let` seems like a closer fit.
Oh yes, I hadn't noticed `def` appeared in the `objects` category //as well!// OK, I'll put `defvar` there, alongside `let`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D72693





More information about the llvm-commits mailing list