[PATCH] D100616: [clang] Fix a potential assert failure

Thu Apr 15 21:05:49 PDT 2021

MaskRay added a comment.

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

> In D100616#2693595 <https://reviews.llvm.org/D100616#2693595>, @MaskRay wrote:
>
>> SVE::FirstTSBuiltin is 8148, the largest.
>
> Isn't SVE::FirstTSBuiltin used to start AArch64's builtins list. So shouldn't AArch64::LastTSBuiltin be larger?

You are right. I misspoke. AArch64::LastTSBuiltin is 8328.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D100616