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

[Ben Shi via Phabricator via cfe-commits]

benshi001 marked an inline comment as done.
benshi001 added inline comments.


================
Comment at: clang/include/clang/Basic/TargetBuiltins.h:334
   static constexpr uint64_t LargestBuiltinID = std::max<uint64_t>(
       {NEON::FirstTSBuiltin, ARM::LastTSBuiltin, SVE::FirstTSBuiltin,
        AArch64::LastTSBuiltin, BPF::LastTSBuiltin, PPC::LastTSBuiltin,
----------------
MaskRay wrote:
> SVE::FirstTSBuiltin can be dropped.
SVE and NEON are removed, VE and RISCV are added.

Can I commit it?


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

https://reviews.llvm.org/D100616

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits