labath added a comment. The fix looks obviously correct, but could you please also add a unit test for this. I guess this would manifest itself as the inability to roundtrip e.g. `std::numeric_limits<long long>::max()` to a Scalar and back.
It's not required, but if you're interested, it would be nice to change all of these assignment operators to do `return *this = Scalar(v);`. That would remove this class of bugs (constructor/operator= inconsistencies) completely. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D80990/new/ https://reviews.llvm.org/D80990 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
