CaseyCarter added inline comments.
================ Comment at: test/std/containers/unord/unord.map/unord.map.cnstr/deduct.pass.cpp:70 +#pragma warning(disable: 4244) // '%s': conversion from '%s' to '%s', possible loss of data +#endif // TEST_COMPILER_C1XX + ---------------- Quuxplusone wrote: > Alternatively, if you wanted to change the instances of `hash<short>` to > `hash<>`, that would be fine with me. (Just as long as it's something > distinguishable from the default of `hash<int>`.) I do like the idea of avoiding rather than suppressing the warning. I'll change these tests to use `hash<long long>` instead of `hash<short>`, which avoids truncating conversions and preserves the "neither `int` nor `long`" property which provides some assurance the deduction guides aren't getting the type from the key type or mapped type. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D68681/new/ https://reviews.llvm.org/D68681 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits