[PATCH] D81583: Update SystemZ ABI to handle C++20 [[no_unique_address]] attribute
Ulrich Weigand via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Jul 8 07:08:37 PDT 2020
uweigand marked 2 inline comments as done.
uweigand added a comment.
In D81583#2137277 <https://reviews.llvm.org/D81583#2137277>, @efriedma wrote:
> I'm tempted to say this is a bugfix for the implementation of no_unique_address, and just fix it globally for all ABIs. We're never going to get anything done here if we require a separate patch for each ABI variant clang supports.
Well, I can certainly do that. Would you prefer me to completely remove the AllowNoUniqueAddress parameter, or keep it but default to true (so ABIs can still override if necessary)?
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D81583/new/
https://reviews.llvm.org/D81583
More information about the cfe-commits
mailing list