[PATCH] D118445: [clang][AVR] Set '-fno-use-cxa-atexit' to default

2022-01-29 Thread Ben Shi via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG653836251ac4: [clang][AVR] Set '-fno-use-cxa-atexit' to default (authored by benshi001). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST A

[PATCH] D118445: [clang][AVR] Set '-fno-use-cxa-atexit' to default

2022-01-29 Thread Fangrui Song via Phabricator via cfe-commits
MaskRay accepted this revision. MaskRay added a comment. This revision is now accepted and ready to land. LGTM. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D118445/new/ https://reviews.llvm.org/D118445 ___ cfe-commits mailing list cfe-commit

[PATCH] D118445: [clang][AVR] Set '-fno-use-cxa-atexit' to default

2022-01-29 Thread Ben Shi via Phabricator via cfe-commits
benshi001 added a comment. fixes https://github.com/llvm/llvm-project/issues/50631 CHANGES SINCE LAST ACTION https://reviews.llvm.org/D118445/new/ https://reviews.llvm.org/D118445 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://list

[PATCH] D118445: [clang][AVR] Set '-fno-use-cxa-atexit' to default

2022-01-29 Thread Ben Shi via Phabricator via cfe-commits
benshi001 added a comment. AVR is baremetal environment, it does not support __cxa_atexit(). CHANGES SINCE LAST ACTION https://reviews.llvm.org/D118445/new/ https://reviews.llvm.org/D118445 ___ cfe-commits mailing list cfe-commits@lists.llvm.org ht

[PATCH] D118445: [clang][AVR] Set '-fno-use-cxa-atexit' to default

2022-01-29 Thread Ben Shi via Phabricator via cfe-commits
benshi001 added a comment. AVR is baremetal environment, it does not support __cxa_atexit() CHANGES SINCE LAST ACTION https://reviews.llvm.org/D118445/new/ https://reviews.llvm.org/D118445 ___ cfe-commits mailing list cfe-commits@lists.llvm.org htt

[PATCH] D118445: [clang][AVR] Set '-fno-use-cxa-atexit' to default

2022-01-29 Thread Ben Shi via Phabricator via cfe-commits
benshi001 added a comment. This patch fixes https://github.com/llvm/llvm-project/issues/43443 https://github.com/llvm/llvm-project/issues/50631 CHANGES SINCE LAST ACTION https://reviews.llvm.org/D118445/new/ https://reviews.llvm.org/D118445 ___ cf

[PATCH] D118445: [clang][AVR] Set '-fno-use-cxa-atexit' to default

2022-01-29 Thread Ben Shi via Phabricator via cfe-commits
benshi001 updated this revision to Diff 404244. benshi001 retitled this revision from "[clang][AVR] Fix a crash in AVRTargetCodeGenInfo::getGlobalVarAddressSpace" to "[clang][AVR] Set '-fno-use-cxa-atexit' to default". benshi001 edited the summary of this revision. CHANGES SINCE LAST ACTION ht