[Bazel] Add support for s390x build target

[vibhutisawant]

Summary

While executing the test suite for Tensorflow(v2.8.0), we encountered multiple TC failures with the below error

'z14' is not a recognized processor for this target

This patch adds the s390x target to the build target list. It fixes TC failures in multiple modules of Tensorflow on s390x arch. It is also tested to have no effect on x86 machines.

[GMNGeoffrey]

Let me know if you need me to land this for you (see https://llvm.org/docs/DeveloperPolicy.html#obtaining-commit-access)

[vibhutisawant]

Yes, could you please commit the same? Author details are "Vibhuti Sawant <[e-mail]>". Thanks.

[GMNGeoffrey]

Recreating the patch with appropriate metadata

[GMNGeoffrey]

However you created this patch, it's causing me issues when trying to arc patch it. It looks like it's missing a lot of a metadata (in particular, a base revision). I've manually recreated the revision and reuploaded it here, but FYI for future reference

[vibhutisawant]

Hi @GMNGeoffrey sorry for the trouble, I uploaded the patch using GUI, and possibly something might have been missed. Will take care next time. Thanks :)

[GMNGeoffrey]

In D125096#3508104, @vibhutisawant wrote:

Hi @GMNGeoffrey sorry for the trouble, I uploaded the patch using GUI, and possibly something might have been missed. Will take care next time. Thanks :)

No worries :-) Just a heads up. I don't understand phab well enough to say what the actual issue is, but I think it's a common problem with the web UI that you might end up with a patch with less metadata. One thing you can do that's helpful if still using the web UI is to generate your patches with full context so people can still see the surrounding code. I think in this case you also didn't set the repository field either. Not sure if either of those would've helped with the base revision thing though.

[vibhutisawant]

Noted ✔️
Yes right, I think I missed setting the repository field. Will make sure to follow above points before I submit a patch. Thanks once again! :)

utils/bazel/llvm-project-overlay/llvm/config.bzl

 })
 
 # TODO: We should split out host vs. target here.
 llvm_config_defines = os_defines + select({
     "@bazel_tools//src/conditions:windows": native_arch_defines("X86", "x86_64-pc-win32"),
     "@bazel_tools//src/conditions:darwin_arm64": native_arch_defines("AArch64", "arm64-apple-darwin"),
     "@bazel_tools//src/conditions:darwin_x86_64": native_arch_defines("X86", "x86_64-unknown-darwin"),
     "@bazel_tools//src/conditions:linux_aarch64": native_arch_defines("AArch64", "aarch64-unknown-linux-gnu"),
+    "@bazel_tools//src/conditions:linux_s390x": native_arch_defines("SystemZ", "systemz-unknown-linux_gnu"),
     "//conditions:default": native_arch_defines("X86", "x86_64-unknown-linux-gnu"),
 }) + [
     # These shouldn't be needed by the C++11 standard, but are for some
     # platforms (e.g. glibc < 2.18. See
     # https://sourceware.org/bugzilla/show_bug.cgi?id=15366). These are also
     # included unconditionally in the CMake build:
     # https://github.com/llvm/llvm-project/blob/cd0dd8ece8e/llvm/cmake/modules/HandleLLVMOptions.cmake#L907-L909
     "__STDC_LIMIT_MACROS",
     "__STDC_CONSTANT_MACROS",
     "__STDC_FORMAT_MACROS",
 ]