This is an archive of the discontinued LLVM Phabricator instance.

Differential D141598

[clang] Redefine some AVR specific macros
ClosedPublic

Authored by benshi001 on Jan 12 2023, 4:03 AM.

Details

Reviewers
aykevl
Miss_Grape
Commits
rG16f9451b0761: [clang] Redefine some AVR specific macros
Summary

fixes https://github.com/llvm/llvm-project/issues/58855

Diff Detail

Event Timeline

benshi001 created this revision.Jan 12 2023, 4:03 AM
Herald added a project: Restricted Project. · View Herald TranscriptJan 12 2023, 4:03 AM
Herald added subscribers: Jim, dylanmckay. · View Herald Transcript
benshi001 requested review of this revision.Jan 12 2023, 4:03 AM
Herald added a project: Restricted Project. · View Herald TranscriptJan 12 2023, 4:03 AM
Herald added subscribers: cfe-commits, jacquesguan. · View Herald Transcript
benshi001 edited the summary of this revision. (Show Details)Jan 12 2023, 4:04 AM
Harbormaster completed remote builds in B207361: Diff 488594.Jan 12 2023, 6:33 AM
Miss_Grape accepted this revision.Jan 12 2023, 7:23 PM
This revision is now accepted and ready to land.Jan 12 2023, 7:23 PM
Closed by commit rG16f9451b0761: [clang] Redefine some AVR specific macros (authored by benshi001). · Explain WhyJan 13 2023, 1:22 AM
This revision was automatically updated to reflect the committed changes.
benshi001 added a commit: rG16f9451b0761: [clang] Redefine some AVR specific macros.

Revision Contents

PathSize
clang/
lib/
Basic/
Targets/
12 lines
test/
Preprocessor/
16 lines

Diff 488898

clang/lib/Basic/Targets/AVR.cpp

Show First 20 Lines • Show All 462 Lines • ▼ Show 20 Lines if (ArchHas3BytePC(Arch)) {
// debatable whether we should be bug-compatible in this case). // debatable whether we should be bug-compatible in this case).
Builder.defineMacro("__AVR_HAVE_EIJMP_EICALL__"); Builder.defineMacro("__AVR_HAVE_EIJMP_EICALL__");
Builder.defineMacro("__AVR_3_BYTE_PC__"); Builder.defineMacro("__AVR_3_BYTE_PC__");
} else { } else {
Builder.defineMacro("__AVR_2_BYTE_PC__"); Builder.defineMacro("__AVR_2_BYTE_PC__");
} }
if (NumFlashBanks >= 1) if (NumFlashBanks >= 1)
Builder.defineMacro("__flash", "__attribute__((address_space(1)))"); Builder.defineMacro("__flash", "__attribute__((__address_space__(1)))");
if (NumFlashBanks >= 2) if (NumFlashBanks >= 2)
Builder.defineMacro("__flash1", "__attribute__((address_space(2)))"); Builder.defineMacro("__flash1", "__attribute__((__address_space__(2)))");
if (NumFlashBanks >= 3) if (NumFlashBanks >= 3)
Builder.defineMacro("__flash2", "__attribute__((address_space(3)))"); Builder.defineMacro("__flash2", "__attribute__((__address_space__(3)))");
if (NumFlashBanks >= 4) if (NumFlashBanks >= 4)
Builder.defineMacro("__flash3", "__attribute__((address_space(4)))"); Builder.defineMacro("__flash3", "__attribute__((__address_space__(4)))");
if (NumFlashBanks >= 5) if (NumFlashBanks >= 5)
Builder.defineMacro("__flash4", "__attribute__((address_space(5)))"); Builder.defineMacro("__flash4", "__attribute__((__address_space__(5)))");
if (NumFlashBanks >= 6) if (NumFlashBanks >= 6)
Builder.defineMacro("__flash5", "__attribute__((address_space(6)))"); Builder.defineMacro("__flash5", "__attribute__((__address_space__(6)))");
} }

clang/test/Preprocessor/predefined-arch-macros.c

Show First 20 Lines • Show All 4,313 Lines • ▼ Show 20 Lines
// CHECK_R600-NOT: #define __HAS_FMAF__ 1 // CHECK_R600-NOT: #define __HAS_FMAF__ 1
// RUN: %clang -march=amdgcn -mcpu=cypress -E -dM %s -o - 2>&1 \ // RUN: %clang -march=amdgcn -mcpu=cypress -E -dM %s -o - 2>&1 \
// RUN: -target r600-unknown-unknown \ // RUN: -target r600-unknown-unknown \
// RUN: | FileCheck -match-full-lines %s -check-prefix=CHECK_R600_FP64 // RUN: | FileCheck -match-full-lines %s -check-prefix=CHECK_R600_FP64
// CHECK_R600_FP64-DAG: #define __R600__ 1 // CHECK_R600_FP64-DAG: #define __R600__ 1
// CHECK_R600_FP64-DAG: #define __HAS_FMAF__ 1 // CHECK_R600_FP64-DAG: #define __HAS_FMAF__ 1
// Begin avr tests ----------------
// RUN: %clang --target=avr -mmcu=atmega328 -E -dM %s -o - 2>&1 \
// RUN: | FileCheck -match-full-lines %s -check-prefix=CHECK-AVR0
// CHECK-AVR0: #define __AVR_ARCH__ 5
// CHECK-AVR0: #define __AVR_ATmega328__ 1
// CHECK-AVR0: #define __flash __attribute__((__address_space__(1)))
// RUN: %clang --target=avr -mmcu=atmega2560 -E -dM %s -o - 2>&1 \
// RUN: | FileCheck -match-full-lines %s -check-prefix=CHECK-AVR1
// CHECK-AVR1: #define __AVR_ARCH__ 6
// CHECK-AVR1: #define __AVR_ATmega2560__ 1
// CHECK-AVR1: #define __flash __attribute__((__address_space__(1)))
// CHECK-AVR1: #define __flash1 __attribute__((__address_space__(2)))
// CHECK-AVR1: #define __flash2 __attribute__((__address_space__(3)))
// CHECK-AVR1: #define __flash3 __attribute__((__address_space__(4)))
// Begin M68k tests ---------------- // Begin M68k tests ----------------
// RUN: %clang -mcpu=68000 -E -dM %s -o - 2>&1 \ // RUN: %clang -mcpu=68000 -E -dM %s -o - 2>&1 \
// RUN: -target m68k-unknown-linux \ // RUN: -target m68k-unknown-linux \
// RUN: | FileCheck -match-full-lines %s -check-prefix=CHECK_M68K_68000_ATOMICS // RUN: | FileCheck -match-full-lines %s -check-prefix=CHECK_M68K_68000_ATOMICS
// RUN: %clang -mcpu=68010 -E -dM %s -o - 2>&1 \ // RUN: %clang -mcpu=68010 -E -dM %s -o - 2>&1 \
// RUN: -target m68k-unknown-linux \ // RUN: -target m68k-unknown-linux \
// RUN: | FileCheck -match-full-lines %s -check-prefix=CHECK_M68K_68000_ATOMICS // RUN: | FileCheck -match-full-lines %s -check-prefix=CHECK_M68K_68000_ATOMICS
▲ Show 20 Lines • Show All 61 Lines • Show Last 20 Lines