This revision was automatically updated to reflect the committed changes.
Closed by commit rGb2638a7a34d9: [clang] Do not extend i8 return values to i16
on AVR. (authored by benshi001).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D139908/new/
Miss_Grape added a comment.
LGTM
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D139908/new/
https://reviews.llvm.org/D139908
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
benshi001 added a comment.
On 32-bit / 64-bit machines, a function returning lower types (via a register)
should an explicit sign/zero extension to full fill the register.
But on AVR, a 8-bit register is used to return 8-bit values, so the sign/zero
extension is unnecessary, and lead to extra
benshi001 added a comment.
This patch fixes https://github.com/llvm/llvm-project/issues/58877 .
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D139908/new/
https://reviews.llvm.org/D139908
___
cfe-commits
benshi001 created this revision.
benshi001 added reviewers: aykevl, dylanmckay.
Herald added a subscriber: Jim.
Herald added a project: All.
benshi001 requested review of this revision.
Herald added subscribers: cfe-commits, jacquesguan.
Herald added a project: clang.
Repository:
rG LLVM Github