[clang] [llvm] [clangd] Add support for the c2000 architecture (PR #125663)

James Nagurne via cfe-commits cfe-commits at lists.llvm.org
Tue Feb 4 11:37:38 PST 2025


DragonDisciple wrote:

General comments:

- Please do not use tabs for spacing. This is what causes the unexpected whitespace gaps in the diff.
- There is a utility, clang-format-diff.py, which will assist in appeasing the code formatting check. However, I noticed running it myself that it changes some entire lists that should likely stay the way they are, so be careful!
  - My usual command line is something like: git diff -U0 --no-color --relative HEAD~1..HEAD | /path/to/clang-format-diff.py -p1 -i -binary /path/to/clang-format(.exe)
- Is there no desire to catch and support C2000 built-in functions? Those will show up in clangd as undefined symbols and will be errors in c99 or later, and in all C++ modes.
- Options
  - The options parser for clang and for the TI compilers are obviously very different. Some options interact with others for the purposes of ease-of-use and error checking, which is something that the clang options parser won't understand. I'll do my best to be diligent in case I have to point this out
  - There are many more options than the ones currently captured. Is it important we get as many as we can? Otherwise this will be a PR that is tailor-fit to a single use-case.

https://github.com/llvm/llvm-project/pull/125663


More information about the cfe-commits mailing list