Fix test outputting to test dir

[mtrofin]

The test introduced in #171118 has llc inadvertently producing an output into the same dir as the test file itself. Most build bots don't clean up the local git repo, which is assumed to not be written by build + test, and patch on top (for build performance reasons), which means the produced output from the aforementioned PR is treated as a test from here onwards, by all bots. Since it's missing RUN lines, we get errors, for example  https://lab.llvm.org/buildbot/#/builders/108/builds/20674

This patch fixes the llc line and also removes the .s. This avoids all bot maintainers go restart their bots. Then, the cleanup is removed in #171256.

[mtrofin]

• Remove cleanup of incorrect output in test dir #171256
• Fix test outputting to test dir #171255 👈 (View in Graphite)
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

[nocchijiang]

Sorry for breaking the builds, and thanks for the fix. I thought piping to FileCheck wasn't necessary since the test doesn't use CHECK lines. It appears FileCheck handles the temporary file deletion.

[mtrofin]

Sorry for breaking the builds, and thanks for the fix. I thought piping to FileCheck wasn't necessary since the test doesn't use CHECK lines. It appears FileCheck handles the temporary file deletion.

No worries, it happens!

btw, it's not FileCheck. Rather, if you:

llc < %s -> this will produce the output to stdout (...and this would have also been a fix)
llc %s -> this, by default, will produce an output next to the input (...and probably you meant to do the above but missed the <)