[PATCH] D139908: [clang] Do not extend i8 return values to i16 on AVR.

2022-12-21 Thread Ben Shi via Phabricator via cfe-commits
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/

[PATCH] D139908: [clang] Do not extend i8 return values to i16 on AVR.

2022-12-20 Thread LiqinWeng via Phabricator via cfe-commits
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

[PATCH] D139908: [clang] Do not extend i8 return values to i16 on AVR.

2022-12-20 Thread Ben Shi via Phabricator via cfe-commits
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

[PATCH] D139908: [clang] Do not extend i8 return values to i16 on AVR.

2022-12-12 Thread Ben Shi via Phabricator via cfe-commits
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

[PATCH] D139908: [clang] Do not extend i8 return values to i16 on AVR.

2022-12-12 Thread Ben Shi via Phabricator via 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