[PATCH] D14755: Handle ARMv6-J as an alias, instead of fake architecture

[Meador Inge via llvm-commits]

meadori added a subscriber: meadori.
meadori added a comment.

It is my understanding as well that `-march=armv6` and `-march=armv6j` are treated the same code gen wise by GCC.  However, they do cause different builtin macro to be defined.  For example:

#define __ARM_ARCH_6J__ 1

and:

#define __ARM_ARCH_6__ 1

I don't know what the current behavior of Clang/LLVM is with respect to these macros, whether this change to unify them will impact that, or if anyone out there has code that will break due to the macros, but it is something to consider (I would probably err on the side of caution, but maybe that is too conservative).


Repository:
  rL LLVM

http://reviews.llvm.org/D14755