[PATCH] D98889: [clang] Replaced some manual pointer tagging with llvm::PointerIntPair.

Josh Haberman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Mar 18 12:41:38 PDT 2021


haberman created this revision.
haberman added a reviewer: rsmith.
haberman requested review of this revision.
Herald added a project: clang.
Herald added a subscriber: cfe-commits.

There is no functional change here (hence no new tests). The only change
is to replace a couple uintptr_t members with llvm::PointerIntPair<> to
clean up the code, making it more readable and less error prone.

This cleanup highlighted that the old code was effectively casting away
const. This is fixed by changing some function signatures.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D98889

Files:
  clang/include/clang/Sema/Initialization.h
  clang/include/clang/Sema/Sema.h
  clang/lib/Sema/SemaDeclCXX.cpp
  clang/lib/Sema/SemaInit.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D98889.331658.patch
Type: text/x-patch
Size: 5909 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20210318/9eb48126/attachment-0001.bin>


More information about the cfe-commits mailing list