This is an archive of the discontinued LLVM Phabricator instance.

[LLDBSwigPythonCallTypeScript] Remove redundant call to type summary func.
ClosedPublic

Authored by sivachandra on Jun 22 2015, 5:51 PM.

Diff Detail

Event Timeline

sivachandra retitled this revision from to [LLDBSwigPythonCallTypeScript] Remove redundant call to type summary func..
sivachandra updated this object.
sivachandra edited the test plan for this revision. (Show Details)
sivachandra added a subscriber: Unknown Object (MLST).

ping.
This was probably missed due to the discussion on the other two patches.

clayborg accepted this revision.Jun 25 2015, 10:04 AM
clayborg edited edge metadata.

I am ok as long as Enrico is ok with it.

This revision is now accepted and ready to land.Jun 25 2015, 10:04 AM
granata.enrico accepted this revision.Jun 25 2015, 10:20 AM
granata.enrico edited edge metadata.

Just discovered the checkmark. As I said, that's a good change, so yes definitely good to go. I assume you ran the test suite and there were no regressions.

No regressions on Linux. Will put it in shortly.

This revision was automatically updated to reflect the committed changes.