[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