[PATCH] D117854: [RISCV] Refactor Zve* extensions.

Yueh-Ting Chen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 21 09:24:31 PST 2022


eopXD 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 see. I will change my patch and see what I should do to address your comments.
Thank you.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D117854



More information about the llvm-commits mailing list