<div dir="auto">The alignment check can just be removed. </div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon., May 27, 2019, 7:57 p.m. Ulrich Weigand via Phabricator, <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">uweigand added a comment.<br>
<br>
Looks like this test is failing on SystemZ since it was added, making all our build bots red:<br>
<br>
  /home/uweigand/sandbox/buildbot/clang-s390x-linux/llvm/tools/clang/test/CodeGenCXX/builtin_FUNCTION.cpp:9:11: error: CHECK: expected string not found in input<br>
  // CHECK: @[[EMPTY_STR:.+]] = private unnamed_addr constant [1 x i8] zeroinitializer, align 1<br>
            ^<br>
  /home/uweigand/sandbox/buildbot/clang-s390x-linux/stage1/tools/clang/test/CodeGenCXX/Output/builtin_FUNCTION.cpp.tmp.ll:1:1: note: scanning from here<br>
  ; ModuleID = '/home/uweigand/sandbox/buildbot/clang-s390x-linux/llvm/tools/clang/test/CodeGenCXX/builtin_FUNCTION.cpp'<br>
  ^<br>
  /home/uweigand/sandbox/buildbot/clang-s390x-linux/stage1/tools/clang/test/CodeGenCXX/Output/builtin_FUNCTION.cpp.tmp.ll:8:1: note: possible intended match here<br>
  @.str = private unnamed_addr constant [1 x i8] zeroinitializer, align 2<br>
  ^<br>
<br>
The problem is that string constants are 2-aligned according to the SystemZ ABI (this is a bit different, but deliberate in order to allow computing their addresses with a LARL instruction).  This makes all the "align 1" checks fail.<br>
<br>
Is this test deliberately verifying the alignment, or could this just be removed?<br>
<br>
<br>
Repository:<br>
  rC Clang<br>
<br>
CHANGES SINCE LAST ACTION<br>
  <a href="https://reviews.llvm.org/D37035/new/" rel="noreferrer noreferrer" target="_blank">https://reviews.llvm.org/D37035/new/</a><br>
<br>
<a href="https://reviews.llvm.org/D37035" rel="noreferrer noreferrer" target="_blank">https://reviews.llvm.org/D37035</a><br>
<br>
<br>
<br>
</blockquote></div>