diff options
author | Fangrui Song <maskray@google.com> | 2019-03-12 14:20:22 +0000 |
---|---|---|
committer | Fangrui Song <maskray@google.com> | 2019-03-12 14:20:22 +0000 |
commit | b1dfbebe8bfbeffcb389d941a370a89a161eb4da (patch) | |
tree | d1b4df0d0cbb5debb7a38495046a8d72c2372732 /lldb/packages/Python/lldbsuite/test/python_api/frame/TestFrames.py | |
parent | 06bee01d2bc7d17e3ee378a1791946b81e2a4611 (diff) | |
download | bcm5719-llvm-b1dfbebe8bfbeffcb389d941a370a89a161eb4da.tar.gz bcm5719-llvm-b1dfbebe8bfbeffcb389d941a370a89a161eb4da.zip |
[SimplifyLibCalls] Simplify optimizePuts
The code might intend to replace puts("") with putchar('\n') even if the
return value is used. It failed because use_empty() was used to guard
the whole block. While returning '\n' (putchar('\n')) is technically
correct (puts is only required to return a nonnegative number on
success), doing this looks weird and there is really little benefit to
optimize puts whose return value is used. So don't do that.
llvm-svn: 355921
Diffstat (limited to 'lldb/packages/Python/lldbsuite/test/python_api/frame/TestFrames.py')
0 files changed, 0 insertions, 0 deletions