<div dir="ltr">* The alignment check can be removed, and I'll do so tomorrow if someone doesn't beat me to it.</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, May 28, 2019 at 1:54 AM Eric Fiselier <<a href="mailto:eric@efcs.ca">eric@efcs.ca</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><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" target="_blank">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);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>
</blockquote></div>