[lit] Try again to fix new tests that fail on Windows bots
authorJoel E. Denny <jdenny.ornl@gmail.com>
Sat, 12 Oct 2019 14:58:43 +0000 (14:58 +0000)
committerJoel E. Denny <jdenny.ornl@gmail.com>
Sat, 12 Oct 2019 14:58:43 +0000 (14:58 +0000)
commit1f5823b788037b0dc8c846a22ef6a5a8a0714af7
treec617df3b217ad51b31a7f7a3b7f26d6544f19dc9
parent0e22cb6ce39b139d35118a7b198390a4e536f0a0
[lit] Try again to fix new tests that fail on Windows bots

Based on the bot logs, when lit's internal diff runs on Windows, it
looks like binary diffs must be decoded also for Python 2.7.
Otherwise, writing the diff to stdout fails with:

```
UnicodeEncodeError: 'ascii' codec can't encode characters in position 7-8: ordinal not in range(128)
```

I did not need to decode using Python 2.7.15 under Ubuntu.  When I do
it anyway in that case, `errors="backslashreplace"` fails for me:

```
TypeError: don't know how to handle UnicodeDecodeError in error callback
```

However, `errors="ignore"` works, so this patch uses that, hoping
it'll work on Windows as well.

This patch leaves `errors="backslashreplace"` for Python >= 3.5 as
there's no evidence yet that doesn't work and it produces more
informative binary diffs.  This patch also adjusts some lit tests to
succeed for either error handler.

This patch adjusts changes introduced by D68664.

llvm-svn: 374657
llvm/utils/lit/lit/builtin_commands/diff.py
llvm/utils/lit/tests/shtest-shell.py