[PATCH] D58811: [clang] Fix misuses of char width to char align

Aleksandr Malykh via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Mar 4 20:36:11 PST 2019


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





More information about the cfe-commits mailing list