[PATCH] D143813: [ClangFE] Check that __sync builtins are naturally aligned.

[Steven Johnson via Phabricator via cfe-commits]

srj added a comment.

> (I certainly agree everyone should be using __atomic and not __sync.)

Halide still uses the `__sync` builtins for 32-bit targets, for reasons I don't have at hand now (IIRC there were downstream users with tooling issues, I will re-check). (I don't think we are using `__sync` for 64-bit targets; if we are, that's a mistake on our part.)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D143813