[PATCH] D71499: Add builtins for aligning and checking alignment of pointers and integers

James Clarke via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jan 2 08:04:46 PST 2020


jrtc27 added inline comments.


================
Comment at: clang/docs/LanguageExtensions.rst:2844
+The builtins ``__builtin_align_up``, ``__builtin_align_down``, return their
+first argument aligned up/down to the next multiple of the second argument.
+The builtin ``__builtin_is_aligned`` returns whether the first argument is
----------------
This should clearly state whether an already-aligned argument is left unmodified or incremented/decremented by the alignment amount. As it stands it reads more like such arguments will be modified, but the implementation is to round, ie the former.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D71499





More information about the cfe-commits mailing list