tbaeder added a comment.

In D155568#4542764 <https://reviews.llvm.org/D155568#4542764>, @aaron.ballman 
wrote:

> In D155568#4525488 <https://reviews.llvm.org/D155568#4525488>, @tbaeder wrote:
>
>> This patch is missing the `Ret<PT_Sint32>` part in `InterpBuiltin()` in the 
>> same file. That needs to change to the correct type as well.
>>
>> I can do that in a follow-up commit. @aaron.ballman Can you come up with a 
>> value I can pass to `-triple` so I can actually test that this works?
>
> Oops, sorry on the delayed response, this one slipped under my radar. You 
> should be able to use `avr-unknown-unknown`: https://godbolt.org/z/4TsPW41d5

Thanks, updated with the avr RUN lines.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D155568/new/

https://reviews.llvm.org/D155568

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to