[llvm] Store sysreg names in-line with their descriptors. (PR #119157)

Owen Anderson via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 9 15:08:45 PST 2024


resistor wrote:

> How good is the error checking, if a future RISC-V extension adds a sysreg with a longer name? Do we need to find a place for a build-time error with a nice message?

The error checking seems fine to me:

```
/Users/resistor/llvm-project/build-release/lib/Target/RISCV/RISCVGenSearchableTables.inc:34559:5: error: initializer-string for char array is too long, array size is 32 but initializer has size 49 (including the null terminating character)
 34559 |   { "testtesttesttesttesttesttesttesttesttesttesttest", "testtesttesttesttesttesttesttesttesttesttesttest", "", 0x283,  {} , false }, // 64
       |     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```

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


More information about the llvm-commits mailing list