[PATCH] D117921: Attributes: add a new allocalign() attribute
Nikita Popov via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Jan 21 13:11:45 PST 2022
nikic requested changes to this revision.
nikic added a comment.
This revision now requires changes to proceed.
Might it make more sense to implement this as an argument attribute? Would be a bit simpler I think, and I don't really see a disadvantage.
If you do want this to be a function int attribute, then the parameter reference should be zero-based, not one-based, for consistency with allocsize.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D117921/new/
https://reviews.llvm.org/D117921
More information about the llvm-commits
mailing list