[PATCH] D99790: [CGCall] Annotate `this` argument with alignment

[Alexander Richardson via Phabricator via cfe-commits]

arichardson added inline comments.


================
Comment at: clang/lib/CodeGen/CGCall.cpp:2314
+      llvm::Align Alignment =
+          getNaturalTypeAlignment(ThisTy, /*BaseInfo=*/nullptr,
+                                  /*TBAAInfo=*/nullptr,
----------------
Shouldn't the alignment also be valid for non-zero address spaces?


This reminds me that I need to upstream our local CHERI changes to take the branch for all address spaces with an all-zero null pointer representation.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D99790