craig.topper added inline comments.
================ Comment at: cfe/trunk/lib/Basic/Targets/ARM.cpp:912 + return true; + } case 'U': // a memory reference... ---------------- Is this supposed to fallthrough from 'T' to 'U'? If so can you add an LLVM_FALLTHROUGH Repository: rL LLVM CHANGES SINCE LAST ACTION https://reviews.llvm.org/D60709/new/ https://reviews.llvm.org/D60709 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits