[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