[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