[PATCH] D76182: [AVR] Support aliases in non-zero address space
John McCall via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Apr 6 14:10:40 PDT 2020
rjmccall requested changes to this revision.
rjmccall added a comment.
This revision now requires changes to proceed.
I agree this needs a test case.
================
Comment at: clang/lib/CodeGen/CodeGenModule.cpp:4550
// Create the new alias itself, but don't set a name yet.
+ unsigned AS = cast<llvm::PointerType>(Aliasee->getType())->getAddressSpace();
auto *GA =
----------------
`Aliasee->getType()->getPointerAddressSpace()`
LGTM. If we ever allow functions to declare an explicit address space, we'll need semantic checks that aliases match up, but that's okay.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D76182/new/
https://reviews.llvm.org/D76182
More information about the cfe-commits
mailing list