steakhal added a comment.

In D86295#2519851 <https://reviews.llvm.org/D86295#2519851>, @ASDenysPetrov 
wrote:

> What about this change? Did you make more measurements?

IMO it needs more justification and measurement to land it.

If my measurement was correct then it would decrease the readability of the 
code without any benefit.
It could be the case due to the allocation strategy & alignment stuff that we 
don't gain anything by making its layout more compact.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D86295

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

Reply via email to