lenary added a comment.
LGTM - the big hint is that the new test is handled the same as
`f_empty_double2` as required by the ABI.
================
Comment at: clang/test/CodeGen/riscv32-ilp32d-abi.cpp:36
+
+struct empty_complex { struct {}; double _Complex fc; };
+
----------------
Please can you add a test of `{ struct {}; float _Complex fc;}` too, just for
regression purposes?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D91278/new/
https://reviews.llvm.org/D91278
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
