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

Michael Liao via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Sep 23 10:15:00 PDT 2019


hliao added a comment.

The current BuildAtomicExpr expects the arguments to be in the API order instead of the AST order. If RebuildAtomicExpr uses the same BuildAtomicExpr, it needs to ensure the order of arguments are in API order; otherwise, arguments (especially the one with memory order) will be misplaced.


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