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