[Lldb-commits] [PATCH] D16868: [lldb] Fix invalid shift operator overload in Scalar

Pavel Labath via lldb-commits lldb-commits at lists.llvm.org
Mon Feb 8 07:29:22 PST 2016


labath added a comment.

`Scalar::operator<<=` seems to do a `m_integer <<= ...`. Will it suffer from the same recursion problem?

In any case, I think the implementations of both methods should use the same API (it doesn't matter which one if both work).

Also it would be super great if you could make a tiny test for this, say in `unittests/Core/ScalarTest.cpp`


Repository:
  rL LLVM

http://reviews.llvm.org/D16868





More information about the lldb-commits mailing list