[flang-commits] [flang] [flang][OpenMP] Reassociate ATOMIC update expressions (PR #153488)

[Michael Kruse via flang-commits]

Meinersbur wrote:

I fixed a compilation break in 38853a0146b1e0a7a6557a788aa1671ddaef385e.

I would recommend using `if constexpr ()` instead of tricks with overload. I think `using Id::operator()` is intended as a fallaback when the derived class's method SFINAE fails, but seems only gcc accepts that. With the additional overload with tag, there is no case where overload resolution would fall back to `using Id::operator()`.

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