[flang] [llvm] [flang][OpenMP] Overhaul implementation of ATOMIC construct (PR #137852)

[Tom Eccles via llvm-commits]

================
@@ -16,10 +16,18 @@
 #include "flang/Semantics/openmp-modifiers.h"
 #include "flang/Semantics/tools.h"
 #include "llvm/ADT/STLExtras.h"
+#include "llvm/ADT/StringSwitch.h"
 #include <variant>
 
 namespace Fortran::semantics {
 
+static_assert(std::is_same_v<SomeExpr, evaluate::Expr<evaluate::SomeType>>);
----------------
tblah wrote:

Please could you add a comment to this like explaining some context so that if this ever fails, whoever debugs it knows where to look.

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