aaron.ballman added a comment.

Thank you for the patch! I'm wondering what the goal is for these changes. We 
don't typically add attributes to Clang that don't have any effect unless 
there's a very compelling reason to do so. Are you intending to add semantics 
for these attributes in follow-up patches?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D133853

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to