HomePhabricator

[NFC] Bump ObjCOrBuiltinIDBits to 15

Authored by jfb on Jun 26 2020, 1:48 PM.

Description

[NFC] Bump ObjCOrBuiltinIDBits to 15

We're now hitting this because we're at the limit for number of builtins:

clang/lib/Basic/IdentifierTable.cpp:39:1: error: static_assert failed due to requirement '2 * LargestBuiltinID < (2 << (ObjCOrBuiltinIDBits - 1))' "Insufficient ObjCOrBuiltinID Bits"
static_assert(2 * LargestBuiltinID < (2 << (ObjCOrBuiltinIDBits - 1)),
^             ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

Bump it to 15 so whoever adds a builtin next (as I am, or as anyone else might) doesn't merge conflict over each other.

Details

Committed
jfbJun 26 2020, 1:48 PM
Parents
rGddbdff3accb6: [sve][acle] Recommit https://reviews.llvm.org/D82501
Branches
Unknown
Tags
Unknown

Event Timeline

plotfi added a subscriber: plotfi.Jun 26 2020, 3:11 PM

Nice. I hit this as well. Thanks @jfb