[PATCH] D158045: [clang][SVE] Rename isVLSTBuiltinType, NFC
Paul Walker via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Aug 16 08:47:08 PDT 2023
paulwalker-arm accepted this revision.
paulwalker-arm added a comment.
This revision is now accepted and ready to land.
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.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D158045/new/
https://reviews.llvm.org/D158045
More information about the cfe-commits
mailing list