- Jan 26, 2023
-
-
Matt Arsenault authored
Also add missing assembler test for the valid cases.
-
Craig Topper authored
The format this uses is CA. I think it may have once shared CS and this didn't get renamed when that changed in D54302.
-
Andrew Young authored
Follow up to https://reviews.llvm.org/D142522. I forgot to push the test changes, was caught only in the post-merge builds.
-
NAKAMURA Takumi authored
-
Benjamin Kramer authored
-
Paul Scoropan authored
flang-new encounters an issue where its unable to invoke itself because the install directory was missing from the program paths on AIX. Reviewed By: w2yehia Differential Revision: https://reviews.llvm.org/D140628
-
David Blaikie authored
-
Matt Arsenault authored
implicit-arg-v5-opt.ll already covers this with more cases.
-
Andrew Young authored
This is a follow up to D142182, to expose LocationAttrs through Python. Reviewed By: ftynse Differential Revision: https://reviews.llvm.org/D142522
-
Shilei Tian authored
-
Paul Kirth authored
Reviewed By: brettw Differential Revision: https://reviews.llvm.org/D142577
-
Michael Jones authored
This patch adds the last conversion for scanf, %p. It is set up to match the %p implementation in our printf. Reviewed By: sivachandra, lntue Differential Revision: https://reviews.llvm.org/D142510
-
Michael Jones authored
To add the current position (%n) conversion, some reorganization needed to be done. The "write a number to this pointer using the length modifier" utilities and a couple other shared parsing functions have been moved into converter_utils.h. This made implementing current_pos_converter very simple. Reviewed By: lntue Differential Revision: https://reviews.llvm.org/D142495
-
Paul Kirth authored
Reviewed By: aeubanks Differential Revision: https://reviews.llvm.org/D142579
-
Arthur Lafrance authored
BlockAddress is currently unimplemented in the LLVM dialect of MLIR; when converting to LLVM dialect MLIR from LLVM IR, mlir-translate currently terminates with an "unhandled constant" error message. Instead, this message could be made more specific, to let the user know that the specific issue is that BlockAddress is unimplemented in the LLVM dialect. Differential Revision: https://reviews.llvm.org/D142337
-
Daniel Thornburgh authored
-
Joseph Huber authored
Summary: The logic here is to add the `.cubin` temporary file if we had to create a new filename to handle it. Unfortuantely the logic was wrong because we compare `const char *` values here. This logic seems to have been wrong for some time, but was never noticed since we never used the relocatable output. Fixes https://github.com/llvm/llvm-project/issues/60301
-
Paul Kirth authored
In many cases, we can use an alias to avoid a symbolic relocations, instead of using the public, interposable symbol. When the instrumented function is in a COMDAT, we can use a hidden alias, and still avoid references to discarded sections. Previous versions of this patch allowed the compiler to name the generated alias, but that would only be valid when the functions were local. Since the alias may be used across TUs we use a more deterministic naming convention, and add a ".local" suffix to the alias name just as we do for relative vtables aliases. https://reviews.llvm.org/rG20894a478da224bdd69c91a22a5175b28bc08ed9 removed an incorrect assertion on Mach-O which caused assertion failures in LLD. We prevent duplicate symbols under ThinLTO + PGO + CFI by disabling alias generation when the target function has MD_type metadata used in CFI. Reviewed By: phosek Differential Revision: https://reviews.llvm.org/D137982
-
Daniel Thornburgh authored
Reviewed By: gulfem Differential Revision: https://reviews.llvm.org/D136702
-
Paul Robinson authored
Basically NFC: A TEST/TEST_F/etc that bails out early (usually because setup failed or some other runtime condition wasn't met) generally should use GTEST_SKIP() to report its status correctly, unless it takes steps to report another status (e.g., FAIL()). I did see a handful of tests show up as SKIPPED after this change, which is not unexpected. The status seemed appropriate in all the new cases.
-
Michael Jones authored
The strtointmax tests weren't running because they depend on the same shared logic as strtol tests and the target for that was being defined after the inttypes targets, causing them to be skipped. This patch moves the inttypes test directory later in the order so that the shared tests are defined before inttypes tests try to use them. Reviewed By: sivachandra, lntue Differential Revision: https://reviews.llvm.org/D142515
-
Florian Hahn authored
-
Jon Chesterfield authored
-
Craig Topper authored
This is an alternative to D140382. This moves the first part of runOnce into a new class that can be reused by D129735. This encapsulates the ownership of the worklist, and VRegInfos into the new class. The code for updating the dead lanes stays in the DetectDeadLanes class. The new class is created on the stack during runOnMachineFunction so all the data structures will be deleted after each run. Previously we only cleared them after each run so the memory might have stayed allocated across runs. Except for VRegInfo which was always deleted. Hopefully this allocation change isn't a big deal. Reviewed By: BeMg Differential Revision: https://reviews.llvm.org/D141993
-
Rafael Auler authored
Old code breaks build with libstdc++ with assertions. Fix it.
-
Aaron Ballman authored
This addresses the issue found in: https://lab.llvm.org/buildbot/#/builders/115/builds/41077
-
Stanislav Mekhanoshin authored
These are copied from pseudos automatically. Differential Revision: https://reviews.llvm.org/D142575
-
Elizabeth Andrews authored
This patch uses existing deferred diagnostics framework to emit error for unsupported type __bf16 in device code. Error is not emitted in host code. Differential Revision: https://reviews.llvm.org/D141375
-
rkayaith authored
Currently `PassManager` defaults to being anchored on `builtin.module`. Switching the default makes `PassManager` consistent with `OpPassManager` and avoids the implicit dependency on `builtin.module`. Specifying the anchor op type isn't strictly necessary when using explicit nesting (existing pipelines will continue to work), but I've updated most call sites to specify the anchor since it allows for better error-checking during pipeline construction. Reviewed By: rriddle Differential Revision: https://reviews.llvm.org/D137731
-
Aaron Ballman authored
The link used to go to a page on Chris Lattner's site that has been gone since at least 2015. Replaced the link with one that has similar information to what Chris had. Fixes #60238
-
Denis Nikitin authored
The check incorrectly identified empty() method call in the template class definition as a stand-alone function call. This led to a crash because the checker did not expect empty() function calls without arguments. Fixes: https://github.com/llvm/llvm-project/issues/59487 Reviewed By: cjdb Differential Revision: https://reviews.llvm.org/D142423
-
Joseph Huber authored
Summary: I forgot that this file "borrowed" the source from the other file tree. Fix that.
-
Joseph Huber authored
Clean up this file after changing it in D142568. Depends on D142568 Reviewed By: tianshilei1992 Differential Revision: https://reviews.llvm.org/D142573
-
Joseph Huber authored
Since D137724 and the LLVM 17 release we have updated to CMake version 3.20. This means that `find_package(CUDA)` is officially deprecated and can be replaced with `find_package(CUDAToolkit)` instead. This patch does this and also cleans up a bit of the CMake. Reviewed By: tianshilei1992 Differential Revision: https://reviews.llvm.org/D142568
-
Joseph Huber authored
Since D137724 and the LLVM 17 release we have updated to CMake version 3.20. This means we can use `find_package(CUDAToolkit)` instead as the legacy module is deprecated. Reviewed By: tianshilei1992 Differential Revision: https://reviews.llvm.org/D142570
-
Guozhi Wei authored
This patch implements the enhancement proposed by https://github.com/llvm/llvm-project/issues/59312. Suppose we have following code v0 = load %addr br %LoadBB LoadBB: v1 = load %addr ... PredBB: ... br %cond, label %LoadBB, label %SuccBB SuccBB: v2 = load %addr ... Instruction v1 in LoadBB is partially redundant, edge (PredBB, LoadBB) is a critical edge. SuccBB is another successor of PredBB, it contains another load v2 which is identical to v1. Current GVN splits the critical edge (PredBB, LoadBB) and inserts a new load in it. A better method is move the load of v2 into PredBB, then v1 can be changed to a PHI instruction. If there are two or more similar predecessors, like the test case in the bug entry, current GVN simply gives up because otherwise it needs to split multiple critical edges. But we can move all loads in successor blocks into predecessors. Differential Revision: https://reviews.llvm.org/D141712
-
Amir Ayupov authored
Allow partial name matching wrt LTO suffixes in `function-order` user-supplied function list, the same as permitted by profile matching. Reviewed By: #bolt, rafauler Differential Revision: https://reviews.llvm.org/D142269
-
Amir Ayupov authored
Move individual warnings under verbosity >= 1, print out a warning with aggregate number. Reviewed By: #bolt, rafauler Differential Revision: https://reviews.llvm.org/D142397
-
wren romano authored
The bulk of D142074 seems to have gotten overwritten due to some sort of merge conflict (afaict there's no record of it having been reverted intentionally). So this commit redoes those changes. In addition to the original changes, this commit also: * moves the definition of `getRankedTensorType` (from `Transforms/CodegenUtils.h` to `IR/SparseTensor.h`), so that it can be used by `IR/SparseTensorDialect.cpp`. * adds `getMemRefType` as another abbreviation. Reviewed By: aartbik Differential Revision: https://reviews.llvm.org/D142503
-
Ingo Müller authored
Reviewed By: ingomueller-net Differential Revision: https://reviews.llvm.org/D142554
-