[lldb-mi] Clean up CMICmdArgSet usage.
CMICmdArgSet stores a vector of non-const pointers to the arguments
that it is validating. It owns them and is responsible for deleting
We don't need to pass a const reference to the argument to
CMICmdArgSet::Add and then take the address and const_cast it
when we can just pass the argument pointer in directly.
This lets us remove some noise at every call site for CMICmdArgSet::Add
and then clean up a couple of bits inside CMICmdArgSet to remove
Reviewers: abidh, ki.stfu, domipheus
Differential Revision: http://reviews.llvm.org/D12878