[PATCH] D138295: [clang][TargetInfo] Use LangAS for getPointer{Width,Align}()
Alexander Richardson via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Nov 30 12:27:28 PST 2022
arichardson added inline comments.
================
Comment at: clang/lib/AST/ASTContext.cpp:2228-2230
+ Width = Target->getPointerWidth(
+ LangAS::Default); // C++ 3.9.1p11: sizeof(nullptr_t)
+ Align = Target->getPointerAlign(LangAS::Default); // == sizeof(void*)
----------------
jrtc27 wrote:
> clang-format(?) screwed up this comment, maybe better to just put it all on one line before these rather than as inline?
Thanks, fixed now.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D138295/new/
https://reviews.llvm.org/D138295
More information about the cfe-commits
mailing list