[Lldb-commits] [PATCH] D80995: [Scalar] Fix assignment operator for long long.

2020-06-03 Thread Pavel Labath via Phabricator via lldb-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG798644e0a452: [Scalar] Fix assignment operator for long long. (authored by werat, committed by labath). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[Lldb-commits] [PATCH] D80995: [Scalar] Fix assignment operator for long long.

2020-06-02 Thread Andy Yankovsky via Phabricator via lldb-commits
werat added a comment. In D80995#2068571 , @labath wrote: > Looks good. Given the timings, I am going to assume you are coordinating with > @miko. Thanks! This is very surprising, but we're not coordinating. I've found out about their patch just now.

[Lldb-commits] [PATCH] D80995: [Scalar] Fix assignment operator for long long.

2020-06-02 Thread Pavel Labath via Phabricator via lldb-commits
labath accepted this revision. labath added a subscriber: miko. labath added a comment. This revision is now accepted and ready to land. Looks good. Given the timings, I am going to assume you are coordinating with @miko. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[Lldb-commits] [PATCH] D80995: [Scalar] Fix assignment operator for long long.

2020-06-02 Thread Andy Yankovsky via Phabricator via lldb-commits
werat created this revision. werat added a reviewer: labath. werat added a project: LLDB. werat added a reviewer: teemperor. Assignment operator `operator=(long long)` currently allocates `sizeof(long)`. On some platforms it works as they have `sizeof(long) == sizeof(long long)`, but on others