SuccessChanges

Summary

  1. [lldb] Use SBProcess::Continue instead of 'run' command in TestTargetAPI.py (details)
  2. [NFC][StackSafety] Move out sort from the loop (details)
  3. [NFC][StackSafety] Change map key comparison (details)
Commit bb4efab9a4d9431dbedb27f04249effd0a73812e by Raphael Isemann
[lldb] Use SBProcess::Continue instead of 'run' command in TestTargetAPI.py

This test is flaky on Green Dragon as it often fails when the process state
is "Invalid" in the assert:
    self.assertEqual(process.GetState(), lldb.eStateExited)
It seems this is related to just doing "run" which apparently invalidates
the Target's process in case it's still running and needs to be restarted.
Just doing 'continue' on the process (and ignoring the error in case it already
finished) prevents that and makes this consistently pass for me.

Just pushing this out to get Green Dragon back online.
The file was modifiedlldb/test/API/python_api/target/TestTargetAPI.py (diff)
Commit 0426e28419799c35cf52fe3d773c5bab9928c699 by Vitaly Buka
[NFC][StackSafety] Move out sort from the loop
The file was modifiedllvm/test/Bitcode/thinlto-function-summary-paramaccess.ll (diff)
The file was modifiedllvm/lib/Analysis/StackSafetyAnalysis.cpp (diff)
Commit 4c30d4b4e5f2c2cb80910638e7827f7b85ee5568 by Vitaly Buka
[NFC][StackSafety] Change map key comparison
The file was modifiedllvm/lib/Analysis/StackSafetyAnalysis.cpp (diff)