[PATCH] D71213: [Alignment][NFC] CreateMemSet use MaybeAlign
Guillaume Chatelet via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Dec 12 02:16:19 PST 2019
gchatelet added a comment.
@foad do you have any insights on how to go with the deprecation?
LLVM has this `LLVM_ATTRIBUTE_DEPRECATED` macro, it's convenient to get a warning but it only works when building without `-Wall`.
Bots and in-tree users have it set by default, it's fine as I'll be fixing those but how about out of tree users?
Another option would be to have a comment but then nobody will actually update the code before it breaks.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D71213/new/
https://reviews.llvm.org/D71213
More information about the cfe-commits
mailing list