[clang] [CIR] Upstream __sync_<OP>_and_fetch builtins (PR #168347)

[Andy Kaylor via cfe-commits]

================
@@ -1107,8 +1197,7 @@ RValue CIRGenFunction::emitBuiltinExpr(const GlobalDecl &gd, unsigned builtinID,
     // Finally, store the result using the pointer.
     bool isVolatile =
         resultArg->getType()->getPointeeType().isVolatileQualified();
-    builder.createStore(loc, emitToMemory(arithOp.getResult(), resultQTy),
-                        resultPtr, isVolatile);
+    builder.createStore(loc, arithOp.getResult(), resultPtr, isVolatile);
----------------
andykaylor wrote:

Is this change intentional? It seems unrelated.

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