[PATCH] D73891: [RISCV] Support experimental/unratified extensions

Simon Cook via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Feb 7 17:08:27 PST 2020


simoncook updated this revision to Diff 243325.
simoncook added a comment.

Add support for Z extensions also under this scheme.

In order to support these I've had to tweak the multi-letter extension parsing a little. The net result is that error messages printed regarding version numbers are now consistent between single letter standard extensions and any multi-letter extension.

This also makes a change to the feature names used for multi-letter extensions, previously an architecture like `rv32isxx1_sxx2` would be accepted with the TargetFeatures `sxx1` and `sxx2` being added. Now an error indicating that `sxx` has been used multiple times in the string, with just the feature `sxx` being added.


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

https://reviews.llvm.org/D73891

Files:
  clang/include/clang/Driver/Options.td
  clang/lib/Driver/ToolChains/Arch/RISCV.cpp
  clang/test/Driver/riscv-arch.c

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D73891.243325.patch
Type: text/x-patch
Size: 14747 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20200208/ca5a7b44/attachment-0001.bin>


More information about the cfe-commits mailing list