[PATCH] D99790: [CGCall] Annotate `this` argument with alignment
Alexander Richardson via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Apr 2 02:02:09 PDT 2021
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
More information about the llvm-commits
mailing list