[PATCH] D28344: [AVR] Add support for the full set of inline asm constraints

[Akira Hatanaka via Phabricator via cfe-commits]

ahatanak added a comment.

Test case?



================
Comment at: lib/Basic/Targets.cpp:8506
+      case 'N': // Integer constant (Range: -1)
+        Info.setRequiresImmediate(-1);
+      case 'O': // Integer constant (Range: 8, 16, 24)
----------------
Is this meant to fall through or do you need a "break" here?


https://reviews.llvm.org/D28344