[PATCH] D142327: [clang][RISCV] Fix ABI handling of empty structs with hard FP calling conventions in C++
Alex Bradbury via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Aug 1 05:47:24 PDT 2023
asb updated this revision to Diff 546012.
asb added a comment.
This update includes test coverage for the bug that was fixed in the reverted version of the patch.
Thanks @rogfer01 for the smaller test case. To my surprise, cvise stripped out almost everything and test_s9 in abi-empty-structs.c reliably caused the assert for me.
Re-review appreciated in order to get this landed and ideally cherry-picked for the 17.x.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D142327/new/
https://reviews.llvm.org/D142327
Files:
clang/docs/ReleaseNotes.rst
clang/lib/CodeGen/ABIInfoImpl.cpp
clang/lib/CodeGen/ABIInfoImpl.h
clang/lib/CodeGen/Targets/RISCV.cpp
clang/test/CodeGen/RISCV/abi-empty-structs.c
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D142327.546012.patch
Type: text/x-patch
Size: 10752 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20230801/60d5beb0/attachment-0001.bin>
More information about the cfe-commits
mailing list