FailedChanges

Summary

  1. Removed unused TestBuilders. (details)
Commit 05481260c40e502d68e8d523b66eb8e23641c8b9 by gkistanova
Removed unused TestBuilders.
The file was removedzorg/buildbot/builders/TestBuilders.py

Summary

  1. Skip -fPIE for AMDGPU and HIP toolchain (details)
  2. [AArch64][GlobalISel] Scalarize <2 x s64> G_MUL since we don't have native support for it. (details)
  3. Recommit "[HIP] Change default --gpu-max-threads-per-block value to 1024" (details)
  4. [HIP] Return non-zero value for invalid target ID (details)
  5. BPF: explicitly specify bpfel triple for certain tests (details)
  6. [MLIR][OpenMP] Removed the ambiguity in flush op assembly syntax (details)
  7. [OpenMP][FIX] Verify compatible types for declare variant calls (details)
Commit 10eb3bf2d430825195bf092d3a75c4745d463826 by Yaxun.Liu
Skip -fPIE for AMDGPU and HIP toolchain

AMDGPU toolchain does not support -fPIE, therefore skip it if specified by driver.

Differential Revision: https://reviews.llvm.org/D88425
The file was modifiedclang/lib/Driver/ToolChains/AMDGPU.h
The file was modifiedclang/lib/Driver/ToolChains/HIP.cpp
The file was modifiedclang/lib/Driver/ToolChains/AMDGPU.cpp
The file was addedclang/test/Driver/hip-fpie-option.hip
Commit b9f2b3bc431a0053159939a4132814fb113af131 by Amara Emerson
[AArch64][GlobalISel] Scalarize <2 x s64> G_MUL since we don't have native support for it.

Differential Revision: https://reviews.llvm.org/D88437
The file was modifiedllvm/lib/Target/AArch64/GISel/AArch64LegalizerInfo.cpp
The file was modifiedllvm/include/llvm/CodeGen/GlobalISel/LegalizerInfo.h
The file was modifiedllvm/test/CodeGen/AArch64/GlobalISel/legalize-mul.mir
Commit 187658b8a6112446d9e7797d495bc7542ac83905 by Yaxun.Liu
Recommit "[HIP] Change default --gpu-max-threads-per-block value to 1024"

Recommit 04abbb3a78186aa92809866b43217c32cba90b71
The file was modifiedclang/include/clang/Basic/LangOptions.def
The file was modifiedclang/test/CodeGenCUDA/amdgpu-kernel-attrs.cu
The file was modifiedclang/test/CodeGenCUDA/kernel-amdgcn.cu
The file was modifiedclang/lib/CodeGen/TargetInfo.cpp
Commit 5a3023a91c0ec82e7272569c5ca7a6eb372b129f by Yaxun.Liu
[HIP] Return non-zero value for invalid target ID

This is part of https://reviews.llvm.org/D60620
The file was modifiedclang/test/Driver/hip-invalid-target-id.hip
The file was modifiedclang/lib/Driver/Driver.cpp
Commit ca1ce397acc39f348f4018e446c84a5746fd5e1e by yhs
BPF: explicitly specify bpfel triple for certain tests

Commit 54d9f743c8b0 ("BPF: move AbstractMemberAccess and
PreserveDIType passes to EP_EarlyAsPossible") changed most
of CORE tests with opt run followed by llc and opt requires
the target triple specified in the IR.

There are few tests where little endian and big endian will
report different result and for little endian versions of
tests, "target triple = "bpf"" will produce wrong results
if the test executed in a big endian machine, e.g.
PowerPC big endian machine, since target "bpf" represents
host endian and will resolve to "bpfeb".
The builtbot reported such failures when build-and-run
on a PowerPC big endian machine.

To fix the issue, using "target triple = "bpfel"" instead.
The file was modifiedllvm/test/CodeGen/BPF/CORE/field-reloc-bitfield-2.ll
The file was modifiedllvm/test/CodeGen/BPF/CORE/offset-reloc-fieldinfo-1.ll
The file was modifiedllvm/test/CodeGen/BPF/CORE/intrinsic-fieldinfo-lshift-1.ll
The file was modifiedllvm/test/CodeGen/BPF/CORE/field-reloc-bitfield-1.ll
The file was modifiedllvm/test/CodeGen/BPF/CORE/offset-reloc-fieldinfo-2.ll
Commit f3ead88e9c6c352edb10cba43e62696fc852c7d6 by kirankumar.tp
[MLIR][OpenMP] Removed the ambiguity in flush op assembly syntax

Summary:
========
Bugzilla Ticket No: Bug 46884 [https://bugs.llvm.org/show_bug.cgi?id=46884]

Flush op assembly syntax was ambiguous:

Consider the below test case:
flush operation is not having any arguments.
But the next statement token i.e "%2" is read as the argument for flush operation and then translator issues an error.
***************************************************************
$ cat -n flush.mlir
     1  llvm.func @_QQmain(%arg0: !llvm.i32) {
     2    %0 = llvm.mlir.constant(1 : i64) : !llvm.i64
     3    %1 = llvm.alloca %0 x !llvm.i32 {in_type = i32, name = "a"} : (!llvm.i64) -> !llvm.ptr<i32>
     4    omp.flush
     5    %2 = llvm.load %1 : !llvm.ptr<i32>
     6    llvm.return
     7  }

$ mlir-translate -mlir-to-llvmir flush.mlir
flush.mlir:5:6: error: expected ':'
  %2 = llvm.load %1 : !llvm.ptr<i32>
     ^
***************************************************************

Solution:
=========
Introduced begin ( `(` ) and end token ( `)` ) to determince the begin and end of variadic arguments.

The patch includes code changes and testcase modifications.

Reviewed By: Valentin Clement, Mehdi AMINI

Differential Revision: https://reviews.llvm.org/D88376
The file was modifiedmlir/include/mlir/Dialect/OpenMP/OpenMPOps.td
The file was modifiedmlir/test/Dialect/OpenMP/ops.mlir
The file was modifiedmlir/test/Target/openmp-llvm.mlir
Commit c942095790decf525a445f3bd68fb9bcc9aa43c6 by johannes
[OpenMP][FIX] Verify compatible types for declare variant calls

Especially for templates we need to check at some point if the base
function matches the specialization we might call instead. Before this
lead to the replacement of `std::sqrt(int(2))` calls with one that
converts the argument to a `std::complex<int>`, clearly not the desired
behavior.

Reported as PR47655

Reviewed By: JonChesterfield

Differential Revision: https://reviews.llvm.org/D88384
The file was modifiedclang/lib/Sema/SemaTemplateInstantiateDecl.cpp
The file was modifiedclang/lib/Sema/SemaOpenMP.cpp
The file was addedclang/test/AST/ast-dump-openmp-begin-declare-variant_template_3.cpp

Summary

  1. Removed unused TestBuilders. (details)
Commit 05481260c40e502d68e8d523b66eb8e23641c8b9 by gkistanova
Removed unused TestBuilders.
The file was removedzorg/buildbot/builders/TestBuilders.py