[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