[llvm] fef2dab - Bugix for buildbot failure at commit 43f031d31264d20cfb8f1ebd606c66e57c231d4d

[Hans Wennborg via llvm-commits]

I see my fix didn't work. It failed on this buildbot:
http://lab.llvm.org:8011/builders/clang-ppc64be-linux/builds/46780

/home/buildbots/ppc64be-clang-test/clang-ppc64be/stage1/bin/lli:
error: error creating EE: No available targets are compatible with
triple "x86_64-unknown-unknown"

And it failed on 32-bit Windows as below. I've reverted the change
that introduced the test in 64c23127508 until it can be fixed
properly.

FAIL: LLVM :: ExecutionEngine/MCJIT/cet-code-model-lager.ll (22607 of 36859)
******************** TEST 'LLVM ::
ExecutionEngine/MCJIT/cet-code-model-lager.ll' FAILED
********************
Script:
--
: 'RUN: at line 1';
c:\src\llvm_package_6a800f6f\build32_stage0\bin\lli.exe
-mtriple=i686-pc-windows-msvc-elf -mtriple=x86_64-unknown-unknown
-code-model=large
C:\src\llvm_package_6a800f6f\llvm-project\llvm\test\ExecutionEngine\MCJIT\cet-code-model-lager.ll
> /dev/null
--
Exit Code: 1

Command Output (stdout):
--
$ ":" "RUN: at line 1"
$ "c:\src\llvm_package_6a800f6f\build32_stage0\bin\lli.exe"
"-mtriple=i686-pc-windows-msvc-elf" "-mtriple=x86_64-unknown-unknown"
"-code-model=large"
"C:\src\llvm_package_6a800f6f\llvm-project\llvm\test\ExecutionEngine\MCJIT\cet-code-model-lager.ll"
# command stderr:
lli.exe: for the --mtriple option: may only occur zero or one times!

On Mon, Apr 6, 2020 at 2:24 PM Hans Wennborg <hans at chromium.org> wrote:
>
> What was the bug?
>
> Because now the test fails like this in my 32-bit Windows build:
>
> $ "c:\src\llvm_package_944db8a4\build32_stage0\bin\lli.exe"
> "-mtriple=i686-pc-windows-msvc-elf" "-code-model=large"
> "C:\src\llvm_package_944db8a4\llvm-project\llvm\test\ExecutionEngine\MCJIT\cet-code-model-lager.ll"
> # command stderr:
> Assertion failed: Is64Bit && "Large code model is only legal in 64-bit
> mode.", file C:\src\llvm_package_944db8a4\llvm-project\llvm\lib\Target\X86\X86ISelLowering.cpp,
> line 4212
>
> I committed a patch that passes the triple to lli in
> 6a800f6f622a7ade275fa6cb1ef07803460d8bb3, let's see how that goes.
>
> Also, I noticed this file (and another from your original commit) end
> with -lager.ll. Should it be -larger.ll? or -large.ll?
>
> Thanks,
> Hans
>
> On Fri, Apr 3, 2020 at 7:26 AM Xiang1 Zhang via llvm-commits
> <llvm-commits at lists.llvm.org> wrote:
> >
> >
> > Author: Xiang1 Zhang
> > Date: 2020-04-03T13:25:35+08:00
> > New Revision: fef2dab100dfc7c49ccf0ce2bacea409324b54ba
> >
> > URL: https://github.com/llvm/llvm-project/commit/fef2dab100dfc7c49ccf0ce2bacea409324b54ba
> > DIFF: https://github.com/llvm/llvm-project/commit/fef2dab100dfc7c49ccf0ce2bacea409324b54ba.diff
> >
> > LOG: Bugix for buildbot failure at commit 43f031d31264d20cfb8f1ebd606c66e57c231d4d
> > Author: Xiang1 Zhang <xiang1.zhang at intel.com>
> > Date:   Fri Apr 3 11:25:38 2020 +0800
> >
> >     Enable IBT(Indirect Branch Tracking) in JIT with CET(Control-flow Enforcement Technology)
> >
> > Added:
> >
> >
> > Modified:
> >     llvm/test/ExecutionEngine/MCJIT/cet-code-model-lager.ll
> >
> > Removed:
> >
> >
> >
> > ################################################################################
> > diff  --git a/llvm/test/ExecutionEngine/MCJIT/cet-code-model-lager.ll b/llvm/test/ExecutionEngine/MCJIT/cet-code-model-lager.ll
> > index 782a2b84a844..478683b016d3 100644
> > --- a/llvm/test/ExecutionEngine/MCJIT/cet-code-model-lager.ll
> > +++ b/llvm/test/ExecutionEngine/MCJIT/cet-code-model-lager.ll
> > @@ -1,5 +1,4 @@
> >  ; RUN: %lli -code-model=large %s > /dev/null
> > -target triple = "x86_64-unknown-linux-gnu"
> >
> >  @a = dso_local local_unnamed_addr global i32 1, align 4
> >
> >
> >
> >
> > _______________________________________________
> > llvm-commits mailing list
> > llvm-commits at lists.llvm.org
> > https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits