dylanmckay marked an inline comment as done.
dylanmckay added inline comments.


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


https://reviews.llvm.org/D28344



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to