Build Issue Teensy 3.2: __AVR__ defined?

[/dev]

I got a bug report that seems to indicate that __AVR__ is defined for the Teensy 3.2 builds. This define is not guarded:

#ifdef __AVR__
  #define NEOGPS_PACKED_DATA
#endif

A build for the Due seems to work correctly. Is __AVR__ supposed to be defined? I guess I assumed that it was the indicator for the 8-bit family of processors. This post might be related.

Cheers,
/dev

[PaulStoffregen]

I'm testing with Arduino 1.6.11 and Teensyduino 1.30, using this:

void setup() {
}

void loop() {
  Serial.print("define test: ");
#ifdef __AVR__
  Serial.print("__AVR__ ");
#endif
#ifdef __arm__
  Serial.print("__arm__ ");
#endif
  Serial.println();
  delay(100);
}

Here's what I see in the serial monitor when I upload to a Teensy 3.2.

[/dev]

Here's what I see in the serial monitor when I upload to a Teensy 3.2.

With a little more information, I determined it was an out-of-date header copy in the repo that did not have the guard.

Thanks for checking,
/dev