[flang-commits] [flang] [flang][OpenMP] Move lowering of ATOMIC to separate file, NFC (PR #144960)

[Omair Javaid via flang-commits]

omjavaid wrote:

This PR has broken the ninja check-flang tests on windows. It probably wasn't caught because the x64 flang buildbot is down and the arm64 buildbot somehow didn't pick it up right away.

The arm64 windows buildbot is now failing these tests as a consequence of this change:

FAIL: Flang::atomic-capture-complex.f90

FAIL: Flang::atomic-capture.f90

To avoid more changes getting piled on top, I'm going to revert these patches for now, since it's been a while since this was committed.

Here's the first build where it failed on arm64: https://lab.llvm.org/buildbot/#/builders/207/builds/2919. For some unexplained reason, it doesn't list the responsible commits. I've run a clean top-of-tree llvm flang build on both arm64 and x64 windows and can confirm the failures are happening on both architectures.

https://github.com/llvm/llvm-project/pull/144960