[PATCH] D138125: [clang] Fix wrong ABI on AVRTiny

2022-11-16 Thread Ben Shi via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG84ef72357328: [clang] Fix wrong ABI of AVRTiny. (authored by benshi001). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D138125/new/

[PATCH] D138125: [clang] Fix wrong ABI on AVRTiny

2022-11-16 Thread Ben Shi via Phabricator via cfe-commits
benshi001 marked an inline comment as done. benshi001 added inline comments. Comment at: clang/test/CodeGen/avr/struct.c:37 +long long fooi64(void) { + return 0xaa5533; +} aykevl wrote: > benshi001 wrote: > > This file should be renamed to return.c instead of

[PATCH] D138125: [clang] Fix wrong ABI on AVRTiny

2022-11-16 Thread Ben Shi via Phabricator via cfe-commits
benshi001 updated this revision to Diff 475808. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D138125/new/ https://reviews.llvm.org/D138125 Files: clang/lib/CodeGen/TargetInfo.cpp clang/test/CodeGen/avr/return-value.c clang/test/CodeGen/avr/struct.c Index:

[PATCH] D138125: [clang] Fix wrong ABI on AVRTiny

2022-11-16 Thread Ayke via Phabricator via cfe-commits
aykevl added inline comments. Comment at: clang/test/CodeGen/avr/struct.c:37 +long long fooi64(void) { + return 0xaa5533; +} benshi001 wrote: > This file should be renamed to return.c instead of current struct.c, I will > do that in another patch, just let you

[PATCH] D138125: [clang] Fix wrong ABI on AVRTiny

2022-11-16 Thread Ben Shi via Phabricator via cfe-commits
benshi001 added inline comments. Comment at: clang/test/CodeGen/avr/struct.c:37 +long long fooi64(void) { + return 0xaa5533; +} This file should be renamed to return.c instead of current struct.c, I will do that in another patch, just let you see clearly what

[PATCH] D138125: [clang] Fix wrong ABI on AVRTiny

2022-11-16 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. A scalar which exceeds 4