[PATCH] D19770: Add FixedSizeStorage to TrailingObjects; NFC

Faisal Vali via llvm-commits llvm-commits at lists.llvm.org
Sun May 1 13:27:43 PDT 2016


faisalv added a comment.

Thanks for your response.  I don't feel strongly about either of those issues - especially since it seems you considered the options - and have good reasons for choosing the alternative.

I would also favor adding in a comment an example of how users are expected to use this functionality to create such objects on the stack.

Thanks!

P.S. Since you weren't seeing my previous comments emailed to you from phabricator - could you let me know if this one is?


http://reviews.llvm.org/D19770





More information about the llvm-commits mailing list