[PATCH] D50979: Eliminate instances of `EmitScalarExpr(E->getArg(n))` in EmitX86BuiltinExpr().

[Richard Smith - zygoloid via Phabricator via cfe-commits]

rsmith added a comment.

I don't think this is NFC. Testcase:

  long long int a, b, c, d;
  unsigned char f() { return _InterlockedCompareExchange128(&(++a), ++b, ++c, &(++d)); }

Today, Clang increments `a`, `b`, `c`, and `d` twice each in `f()`.


https://reviews.llvm.org/D50979