[PATCH] D151298: [clang][LoongArch] Fix the calling convention for empty struct in C++ mode
Xi Ruoyao via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed May 24 08:00:49 PDT 2023
xry111 added a comment.
If you are really determined to do this, then OK. I'm in a very bad
mood and I don't want to spend my mental strength on debating (esp. on a
corner case unlikely to affect "real" code) anymore.
But remember to add a entry in GCC 14 changes.html, and test this thing:
struct Empty {};
struct Something : Empty
{
double a, b;
};
If we are not careful enough we may introduce a ABI mismatch between -
std=c++14 and -std=c++17 here. See https://gcc.gnu.org/PR94383.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D151298/new/
https://reviews.llvm.org/D151298
More information about the cfe-commits
mailing list