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



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

Reply via email to