jacquesguan added a comment.

In D117854#3261659 <https://reviews.llvm.org/D117854#3261659>, @craig.topper 
wrote:

> In D117854#3261618 <https://reviews.llvm.org/D117854#3261618>, @eopXD wrote:
>
>> A question that bothers me is that we need to specify a specific feature to 
>> enable the TARGET_BUILTIN for rvv, if we are removing the “subset” 
>> relationship here, how can V enable the builtin-s?
>
> If I remember right "experimental-zve32x|experimental-v" should be valid 
> syntax. Though I'm not sure the string is used at all because of how the 
> clang_builtin_alias used by the header works.

I modified the `RISCVISAInfo` and make this revision works for Clang.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D117854

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to