[PATCH] D155568: [clang][Interp] Make sure we push integers of the correct size

Timm Bäder via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Jul 29 02:04:41 PDT 2023


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



More information about the cfe-commits mailing list