amalykh added a comment.

Thank you for your comments, Roman.
Despite the fact, that there are no official LLVM targets that will be 
influenced by this patch, there are architectures which support only word 
addressing mode, for which the assumption of char align equals to 8 is not true.
The patch have been tested for a proprietary architecture, where there is a 
difference between align and width of char. As for the official targets, it's 
more like a style improvement.


Repository:
  rC Clang

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

https://reviews.llvm.org/D58811



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

Reply via email to