Skip to content

Commit

Permalink
Don't define __ARM*__ for Clang
Browse files Browse the repository at this point in the history
I have a patch for Clang ready that I will upstream instead.
  • Loading branch information
Midar committed Nov 8, 2015
1 parent f7ededa commit bfe60c1
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 7 deletions.
5 changes: 1 addition & 4 deletions build/jam/ArchitectureRules
Expand Up @@ -43,10 +43,7 @@ rule ArchitectureSetup architecture

local cpu = $(HAIKU_CPU_$(architecture)) ;
if $(cpu) = arm {
if $(HAIKU_CC_IS_CLANG_$(architecture)) = 1 {
# Clang only defines __ARMEL__ when using -target arm-haiku-unknown
gccBaseFlags += -D__ARM__=1 ;
} else {
if $(HAIKU_CC_IS_CLANG_$(architecture)) != 1 {
# For stackcrawls - not supported by Clang
gccBaseFlags += -mapcs-frame ;
}
Expand Down
3 changes: 0 additions & 3 deletions build/jam/board/rpi2/BoardSetup
Expand Up @@ -59,9 +59,6 @@ HAIKU_BOARD_SDIMAGE_SIZE = 128 ;
#

local flags = -march=armv7-a -mfloat-abi=hard ;
if $(HAIKU_CC_IS_CLANG_$(architecture)) = 1 {
flags += -D__ARM_ARCH__=7 ;
}

HAIKU_ASFLAGS_$(HAIKU_PACKAGING_ARCH) += $(flags) ;
HAIKU_CCFLAGS_$(HAIKU_PACKAGING_ARCH) += $(flags) ;
Expand Down

0 comments on commit bfe60c1

Please sign in to comment.