[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