[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