[PATCH] D67924: [Sema] Fix the atomic expr rebuilding order.

Erich Keane via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Sep 23 10:21:14 PDT 2019


erichkeane added a comment.

Yikes, good catch!

Would we be better off instead to just modify how the other switch loads the value?  Presumably something like, "if (NeedsRearrangeArgs) SubExprs.append(Args.begin(), Args.end()); else /*the switch*/.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D67924/new/

https://reviews.llvm.org/D67924





More information about the cfe-commits mailing list