[Lldb-commits] [PATCH] D44907: Remove Scalar::Cast

2018-04-02 Thread Davide Italiano via Phabricator via lldb-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL328985: [Core] Grab-bag of improvements for Scalar. (authored by davide, committed by ). Herald added a subscriber: llvm-commits. Changed prior to commit: https://reviews.llvm.org/D44907?vs=139844&id=14

[Lldb-commits] [PATCH] D44907: Remove Scalar::Cast

2018-03-29 Thread Davide Italiano via Phabricator via lldb-commits
davide added a comment. OK, I'll commit it for you later today. https://reviews.llvm.org/D44907 ___ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [PATCH] D44907: Remove Scalar::Cast

2018-03-29 Thread Tom Tromey via Phabricator via lldb-commits
tromey added a comment. In https://reviews.llvm.org/D44907#1051931, @davide wrote: > @tromey thanks! do you need somebody to commit this for you? Yes. I do not have commit access. https://reviews.llvm.org/D44907 ___ lldb-commits mailing list lld

[Lldb-commits] [PATCH] D44907: Remove Scalar::Cast

2018-03-29 Thread Davide Italiano via Phabricator via lldb-commits
davide added a comment. @tromey thanks! do you need somebody to commit this for you? https://reviews.llvm.org/D44907 ___ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [PATCH] D44907: Remove Scalar::Cast

2018-03-29 Thread Greg Clayton via Phabricator via lldb-commits
clayborg accepted this revision. clayborg added a comment. This revision is now accepted and ready to land. Very nice. Thanks for doing this. Someone a while back had converted Scalar to use APInt and APFloat and never got around to all of these details. https://reviews.llvm.org/D44907 _