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

Aaron Ballman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jul 28 08:37:26 PDT 2023


aaron.ballman added a comment.

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


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

https://reviews.llvm.org/D155568



More information about the cfe-commits mailing list