https://github.com/SixWeining closed
https://github.com/llvm/llvm-project/pull/70320
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
SixWeining wrote:
> LGTM. I've no permission to make a formal ("GitHub style") approval.
Thanks. You can request to join the llvm organization via
https://llvm.org/docs/DeveloperPolicy.html#obtaining-commit-access firstly.
Then you can make formal ("Github style") approval.
xry111 wrote:
LGTM. I've no permission to make a formal ("GitHub style") approval.
https://github.com/llvm/llvm-project/pull/70320
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/heiher approved this pull request.
https://github.com/llvm/llvm-project/pull/70320
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
SixWeining wrote:
Hi @xen0n @xry111, what do you think about this change? If it is ok, I'd like
to checrry-pick to release/17.x (hope it's not too late...).
https://github.com/llvm/llvm-project/pull/70320
___
cfe-commits mailing list
llvmbot wrote:
@llvm/pr-subscribers-clang-codegen
Author: Lu Weining (SixWeining)
Changes
How empty structs (not as fields of container struct) are passed in C++ is not
explicitly documented in psABI. However, this patch fixes the mismatch with g++.
Note that the unnamed bitfield case
https://github.com/SixWeining created
https://github.com/llvm/llvm-project/pull/70320
How empty structs (not as fields of container struct) are passed in C++ is not
explicitly documented in psABI. However, this patch fixes the mismatch with g++.
Note that the unnamed bitfield case `struct {