unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
* bug#72388: 31.0.50; Use tree-sitter-cuda grammar but with tree-sitter-cpp's font-lock/indentation rules
@ 2024-07-31  0:05 ` Yuan Fu
  2024-08-12 23:10   ` Ergus via Bug reports for GNU Emacs, the Swiss army knife of text editors
  0 siblings, 1 reply; 3+ messages in thread
From: Yuan Fu @ 2024-07-31  0:05 UTC (permalink / raw)
  To: 72388

Originated from "Implementing cuda-ts-mode” on emacs-devel [1]. Basically we want to create a major mode for tree-sitter-cuda. Tree-sitter-cuda grammar is basically cpp grammar plus some cuda rules. So it makes sense to reuse cpp mode’s font-lock/indentation rules. But those rules directs to the ‘cpp grammar. A simple solution is to allow users to map one language to another; then when ever lisp tries to create or get a parser for lang A, lang B is used.

Yuan

[1] Message-ID: <v2a2js5ssoz7d3hquenaieubgmj263tz3ejbogordd5t3ys4v2@efpjojlxrlv4>




^ permalink raw reply	[flat|nested] 3+ messages in thread

* bug#72388: 31.0.50; Use tree-sitter-cuda grammar but with tree-sitter-cpp's font-lock/indentation rules
  2024-07-31  0:05 ` bug#72388: 31.0.50; Use tree-sitter-cuda grammar but with tree-sitter-cpp's font-lock/indentation rules Yuan Fu
@ 2024-08-12 23:10   ` Ergus via Bug reports for GNU Emacs, the Swiss army knife of text editors
  2024-08-20  3:44     ` Yuan Fu
  0 siblings, 1 reply; 3+ messages in thread
From: Ergus via Bug reports for GNU Emacs, the Swiss army knife of text editors @ 2024-08-12 23:10 UTC (permalink / raw)
  To: 72388; +Cc: Yuan Fu

Hi Yuan:

I am just trying to use the new treesit-language-remap-alist and so far
I understand your idea of how to make it work for the basics. But it is
not clear how can I use the new var to add the new syntax entries.

I mean, looking at the grammar:

https://github.com/tree-sitter-grammars/tree-sitter-cuda/blob/master/grammar.js

You can see that it is the inherited C++ syntax, with extra entries in
some fields (like _declaration_modifiers or delete_expression). For this
part the new variable may work perfectly.

However there are a few "new" entries like: kernel_call_expression,
kernel_call_syntax and launch_bounds that will need special handling.

What's your suggestion?

I mean, for the latest what we really need is to extend (partially
redefine) the c-ts-mode--indent-styles, c-ts-mode--keywords and so on.

Is that the intended approach?

Best,
Ergus





^ permalink raw reply	[flat|nested] 3+ messages in thread

* bug#72388: 31.0.50; Use tree-sitter-cuda grammar but with tree-sitter-cpp's font-lock/indentation rules
  2024-08-12 23:10   ` Ergus via Bug reports for GNU Emacs, the Swiss army knife of text editors
@ 2024-08-20  3:44     ` Yuan Fu
  0 siblings, 0 replies; 3+ messages in thread
From: Yuan Fu @ 2024-08-20  3:44 UTC (permalink / raw)
  To: Ergus; +Cc: 72388


Hey Ergus, sorry for the delay.

> On Aug 12, 2024, at 4:10 PM, Ergus via Bug reports for GNU Emacs, the Swiss army knife of text editors <bug-gnu-emacs@gnu.org> wrote:
> 
> Hi Yuan:
> 
> I am just trying to use the new treesit-language-remap-alist and so far
> I understand your idea of how to make it work for the basics. But it is
> not clear how can I use the new var to add the new syntax entries.
> 
> I mean, looking at the grammar:
> 
> https://github.com/tree-sitter-grammars/tree-sitter-cuda/blob/master/grammar.js
> 
> You can see that it is the inherited C++ syntax, with extra entries in
> some fields (like _declaration_modifiers or delete_expression). For this
> part the new variable may work perfectly.
> 
> However there are a few "new" entries like: kernel_call_expression,
> kernel_call_syntax and launch_bounds that will need special handling.
> 
> What's your suggestion?
> 
> I mean, for the latest what we really need is to extend (partially
> redefine) the c-ts-mode--indent-styles, c-ts-mode--keywords and so on.
> 
> Is that the intended approach?

Yeah, you can define your own cuda-indent-styles by grabbing c-ts-mode—indent-styles and adding your rules on top of it, something like

(append your-styles
        (c-ts-mode--indent-styles 'c++))

For font-lock, something like

(append (treesit-font-lock-rules
         :language 'cuda
         :faeture 'xxx
         '(queries))
        (c-ts-mode--font-lock-settings 'c++))

The new treesit-language-remap-alist will ensure that even though the config you borrow from c-ts-mode calls for c++, Emacs will actually use cuda grammar in your major mode.

Yuan




^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2024-08-20  3:44 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <j77vrtrwipskv52jzhoilntlm2gqamvt7ledyeuxkugrsh7ojy.ref@sl3msvhlp73y>
2024-07-31  0:05 ` bug#72388: 31.0.50; Use tree-sitter-cuda grammar but with tree-sitter-cpp's font-lock/indentation rules Yuan Fu
2024-08-12 23:10   ` Ergus via Bug reports for GNU Emacs, the Swiss army knife of text editors
2024-08-20  3:44     ` Yuan Fu

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).