[PATCH] D37310: [Atomic] Merge alignment information from Decl and from Type when emit atomic expression.

Taewook Oh via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 5 09:24:17 PDT 2018


twoh added a comment.
Herald added subscribers: llvm-commits, jfb.

Hello, I observed a case where atomic builtin generates libcall when the corresponding sync builtin generates an atomic instruction (https://bugs.llvm.org/show_bug.cgi?id=38846). It seems that the alignment checking for __atomic builtins (line 759 of this patch) results the difference, and wonder if the check is actually necessary. Could anyone please shed some light on understanding this? Thanks!


Repository:
  rL LLVM

https://reviews.llvm.org/D37310





More information about the llvm-commits mailing list