<div dir="auto">Awesome, thanks! </div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Jun 24, 2021, 11:59 PM Tom Stellard <<a href="mailto:tstellar@redhat.com">tstellar@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 6/24/21 12:17 PM, Armin Hamar wrote:<br>
> I would like to but i don’t have a bugzilla account and requesting seems the be not possible (email doesn’t get send as it’s rejected by the server).<br>
> <br>
> So for me there isn’t really a way to file this in. I did send it directly to <a href="mailto:admin@lists.llvm.org" target="_blank" rel="noreferrer">admin@lists.llvm.org</a> ..<br>
> <br>
<br>
I just had a chance to review the patch, and it is very straightforward,<br>
so I went ahead and backported it.  No need to file a bug now.<br>
<br>
-Tom<br>
<br>
>> Am 24.06.2021 um 20:52 schrieb Tom Stellard <<a href="mailto:tstellar@redhat.com" target="_blank" rel="noreferrer">tstellar@redhat.com</a>>:<br>
>><br>
>> On 6/24/21 10:56 AM, Armin Hamar wrote:<br>
>>> Hello 👋🏻<br>
>>> I have a little problem with the current llvm-c version it seems like it is not possible to use it correctly?<br>
>><br>
>> Can you file a bug for this an put release-12.0.1 in the 'blocks' field.<br>
>><br>
>> -Tom<br>
>><br>
>>> The reason is that byval and sret seem to require a type attribute which can’t be created in the C api. Someone already wrote a fix for it and that is already in the main repo but it didn’t make it into llvm12. This wasn’t a real problem with llvm11 but now is at it seems to be required for functions to actually work.<br>
>>> This is the commit that allows one to make these TypeAttributes:<br>
>>> <a href="https://github.com/llvm/llvm-project/commit/528f6f7d617757addac9b51dd5bcc1ab1352e9be" rel="noreferrer noreferrer" target="_blank">https://github.com/llvm/llvm-project/commit/528f6f7d617757addac9b51dd5bcc1ab1352e9be</a> <<a href="https://github.com/llvm/llvm-project/commit/528f6f7d617757addac9b51dd5bcc1ab1352e9be" rel="noreferrer noreferrer" target="_blank">https://github.com/llvm/llvm-project/commit/528f6f7d617757addac9b51dd5bcc1ab1352e9be</a>><br>
>>> My request is now to add this as a patch to llvm-c for version 12 if this can be done quicker than a new llvm version (e.g 13). The commit also doesn’t edit anything that’s already there but just adds new functions for type attributes. Almost seems like it slipped under the radar.<br>
>>> Best regards,<br>
>>> Armin Hamar<br>
>><br>
> <br>
<br>
</blockquote></div>