[PATCH] D138841: AMDGPU: Fix not emitting code for exotic constructor types
Matt Arsenault via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Nov 29 06:42:41 PST 2022
arsenm added a comment.
In D138841#3956178 <https://reviews.llvm.org/D138841#3956178>, @sameerds wrote:
> The change description says "should" in a couple of places. If we should error out on unexpected entries, then can it be handled in this change itself? The ctor/dtor bodies in the test look well-formed, but do they actually produce the correct code in the ISA? Should we have an llc test for the same or similar input?
We shouldn't error, we certainly have all the handling for handling calls to anything. I can add an extra run line here to be sure
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D138841/new/
https://reviews.llvm.org/D138841
More information about the llvm-commits
mailing list