[PATCH] D77491: [Sema] Introduce BuiltinAttr, per-declaration builtin-ness

John McCall via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Aug 31 14:43:40 PDT 2020


rjmccall added inline comments.


================
Comment at: clang/lib/Serialization/ASTWriter.cpp:3387
     Bits = (Bits << 1) | unsigned(II->isPoisoned());
-    Bits = (Bits << 1) | unsigned(II->hasRevertedBuiltin());
+    Bits <<= 1; // Previously used to indicate reverted builtin.
     Bits = (Bits << 1) | unsigned(II->hasRevertedTokenIDToIdentifier());
----------------
Clang bitcode isn't really a stable format, it's fine to just drop this bit.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D77491



More information about the cfe-commits mailing list