jacquesguan added a comment. In D158045#4592331 <https://reviews.llvm.org/D158045#4592331>, @paulwalker-arm wrote:
> I'd hope there are common code paths where `isVLSTBuiltinType` would still be > useful but I guess this step is required to make that happen anyway. > > If you permit a minor request can the new name please be > `isSveVLSBuiltinType` to match the case used by the related `getSveEltType` > function and the extra `T` in the old name has always bugged me. Yes, I am working on a patch to implement clang vector language support for rvv just like we have on sve. Most part we could reuse `isVLSTBuiltinType` that contains VLST of both sve and rvv. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D158045/new/ https://reviews.llvm.org/D158045 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
