<div dir="ltr">LGTM</div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Mar 30, 2015 at 11:21 AM, Ulrich Weigand <span dir="ltr"><<a href="mailto:Ulrich.Weigand@de.ibm.com" target="_blank">Ulrich.Weigand@de.ibm.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
<br>
Hello,<br>
<br>
SystemZ prefers to align all global variables to two bytes, which is<br>
implemented by setting the TargetInfo member MinGlobalAlign.<br>
<br>
However, for compatibility with existing compilers this should *not*<br>
change the ABI alignment value as retrieved via __alignof__, which<br>
it currently does.<br>
<br>
This patch fixes the issue by having ASTContext::getDeclAlign ignore<br>
the MinGlobalAlign setting in the ForAlignof case.<br>
<br>
Since SystemZ is the only platform setting MinGlobalAlign, this should<br>
cause no change for any other target.<br>
<br>
(See attached file: clang-align-global)<br>
<br>
<br>
Mit freundlichen Gruessen / Best Regards<br>
<br>
Ulrich Weigand<br>
<br>
--<br>
  Dr. Ulrich Weigand | Phone: <a href="tel:%2B49-7031%2F16-3727" value="+497031163727">+49-7031/16-3727</a><br>
  STSM, GNU/Linux compilers and toolchain<br>
  IBM Deutschland Research & Development GmbH<br>
  Vorsitzende des Aufsichtsrats: Martina Koederitz | Geschäftsführung: Dirk<br>
Wittkopp<br>
  Sitz der Gesellschaft: Böblingen | Registergericht: Amtsgericht<br>
Stuttgart, HRB 243294<br>_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
<br></blockquote></div><br></div>