[llvm] [RISCV] Introduce a new tune feature string syntax and its parser (PR #168160)

[Sam Elliott via llvm-commits]

lenary wrote:

I think there's a difficult trade-off here around trying to be clear what clang/LLVM accepts vs having this be reasonably portable between compilers. What effort do we decide is "best".

I am not against warnings, but they do increase the complexity for supporting this, both inside clang and for using this in any build system where a user wants to stay warning free across toolchains. I also don't know how often users will just ignore these warnings when they want to be portable between toolchains.

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