MaskRay accepted this revision. MaskRay added a comment. This revision is now accepted and ready to land.
Thanks! ================ Comment at: clang/lib/Sema/SemaChecking.cpp:14296 + if (SourceBT && SourceBT->isInteger() && TargetBT && + TargetBT->isInteger() && ---------------- Consider adding an example when this condition triggers to help understanding, e.g. `long t2 = 1LL << x;` on LP64 systems. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D144011/new/ https://reviews.llvm.org/D144011 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits