[PATCH] D91467: ADT: Take small enough, trivially copyable T by value in SmallVector

[Mehdi AMINI via Phabricator via llvm-commits]

mehdi_amini added a comment.

Can you make more clear in the description the motivation for this patch? Is this mostly about iterator invalidation? Is this expected to make code safer? 
I'm not sure about making the mental model a bit harder by having different iterator invalidation behavior implicitly based on T, WDYT?


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

https://reviews.llvm.org/D91467