- Oct 27, 2023
-
-
Benjamin Kramer authored
Avoids unused variable warnings in release builds. NFCI.
-
Rainer Orth authored
As discussed in [[Driver] Link Flang runtime on Solaris](https://github.com/llvm/llvm-project/pull/65644), `clang -r` incorrectly passes both `-Bdynamic` and `-e _start` to `ld` which lets the linker choke. This patch fixes this, omitting `-Bdynamic` completely which is the linker default. Tested on `amd64-pc-solaris2.11` and `sparcv9-sun-solaris2.11`.
-
Amara Emerson authored
This change has virtually no code size regressions on the llvm test suite (+ SPECs) while having these improvements (measured with -Os on Darwin arm64): External/S.../CFP2006/450.soplex/450.soplex 214024.00 213920.00 -0.0% External/S...7speed/641.leela_s/641.leela_s 93412.00 93348.00 -0.1% External/S...17rate/541.leela_r/541.leela_r 93412.00 93348.00 -0.1% MultiSourc.../Applications/JM/lencod/lencod 426044.00 425748.00 -0.1% MultiSourc...rks/mediabench/gsm/toast/toast 20436.00 20416.00 -0.1% MultiSourc...ench/telecomm-gsm/telecomm-gsm 20436.00 20416.00 -0.1% MultiSourc...Prolangs-C/assembler/assembler 16172.00 16156.00 -0.1% MultiSourc...nch/mpeg2/mpeg2dec/mpeg2decode 35332.00 35256.00 -0.2% SingleSour...Adobe-C++/stepanov_abstraction 6904.00 6888.00 -0.2% External/SPEC/CINT2000/254.gap/254.gap 366060.00 365132.00 -0.3% MultiSourc...-ProxyApps-C++/PENNANT/PENNANT 79688.00 79484.00 -0.3% External/S...NT2006/464.h264ref/464.h264ref 352044.00 351132.00 -0.3% SingleSour...arks/Adobe-C++/functionobjects 15524.00 15480.00 -0.3% SingleSour...arks/Adobe-C++/stepanov_vector 10728.00 10696.00 -0.3% SingleSour...ks/Misc-C++/stepanov_container 16900.00 16848.00 -0.3% MultiSource/Applications/oggenc/oggenc 124184.00 123780.00 -0.3% SingleSour...tout-C++/Shootout-C++-wordfreq 7060.00 7036.00 -0.3% MultiSourc...ity-rijndael/security-rijndael 8976.00 8936.00 -0.4% MultiSource/Benchmarks/McCat/18-imp/imp 9816.00 9772.00 -0.4% SingleSour...chmarks/Misc-C++/stepanov_v1p2 1772.00 1764.00 -0.5% MultiSourc...iabench/g721/g721encode/encode 5492.00 5464.00 -0.5% MultiSourc...rks/McCat/03-testtrie/testtrie 1364.00 1344.00 -1.5% SingleSour.../execute/GCC-C-execute-pr42833 400.00 364.00 -9.0% Doing so also prevents a regression described in https://reviews.llvm.org/D143624 Differential Revision: https://reviews.llvm.org/D149918
-
Jessica Clarke authored
These are an artifact of how types are structured but serve little purpose, merely showing that the type is sugared in some way. For example, ElaboratedType's existence means struct S gets printed as 'struct S':'struct S' in the AST, which is unnecessary visual clutter. Note that skipping the second print when the types have the same string matches what we do for diagnostics, where the aka will be skipped.
-
Jinsong Ji authored
https://github.com/llvm/llvm-project/pull/69824 added libc build, but missed the folder in ninja command, is causing failures. ninja: fatal: chdir to 'docs-libc-html' - No such file or directory ninja: Entering directory `docs-libc-html'
-
Aart Bik authored
This no longer tests a required feature.
-
Ralf Jung authored
Follow-up to #66579: while implementing those semantics in Miri I realized there's a special case to be considered in truncating float casts.
-
Youngsuk Kim authored
Opaque pointer cleanup effort. NFC.
-
Craig Topper authored
Even though we don't have W instructions for them. This treats them the same as other binary operators.
-
LLVM GN Syncbot authored
-
Aart Bik authored
Various TODOs had been added that actually removed the actual test. This puts the CHECK test backs and removes the TODOs that have no immediate plans.
-
Alpha Abdoulaye authored
Reland https://reviews.llvm.org/D158124 Fixed `-fpermissive` error reported by gcc only.
-
Nishant Patel authored
-
Amara Emerson authored
The pre-index matcher just needs some small heuristics to make sure it doesn't cause regressions. Apart from that it's a simple change, since the only difference is an immediate operand of '1' vs '0' in the instruction.
-
Matt Harding authored
Some textual editing errors got through this pull request that was merged a few weeks ago: https://github.com/llvm/llvm-project/pull/65876 This patch clears up the unintentional duplicated line, and white-space at the end of the lines.
-
Louis Dionne authored
I came across those typedefs while working on another change, and I noticed they were just never used.
-
Ilya Tocar authored
If we know that index is larger than SSO size, we know that we can't be in SSO case, and should access the pointer. This removes extra check from operator[] for inputs known at compile time to be larger than SSO.
-
Abhishek Varma authored
-- In order to compute maximum, we should always initialise the result with the largest negative value possible for the concerned element type, instead of the smallest. -- This commit essentially adds a fix to this issue. Signed-off-by:Abhishek Varma <abhishek@nod-labs.com>
-
Daniel Grumberg authored
For declarations declared inside a macro, e.g.: ``` `#define MAKE_FUNC(suffix) \ /// Not selected doc comment \ void func_##suffix(void) { } /// Doc comment foo MAKE_FUNC(foo) /// Doc comment bar MAKE_FUNC(bar) ```` Prefer the doc comment at the expansion site instead of the one defined in the macro. rdar://113995729 -
David Spickett authored
Since https://github.com/llvm/llvm-project/pull/68052 this has been failing. https://lab.llvm.org/buildbot/#/builders/219/builds/6545 Follow up changes have not fixed it, XFAIL while I debug it.
-
Kiran Chandramohan authored
These are copies of tests in flang/test/Lower/OpenMP/FIR.
-
Aart Bik authored
-
- Oct 26, 2023
-
-
Aaron Ballman authored
Fixes https://github.com/llvm/llvm-project/issues/65858
-
Aviad Cohen authored
linalg::CopyOp is much more generic and useful to promote buffers. In addition, this is linalg transform and makes more sense to use linalg operations when possible.
-
Alexey Bataev authored
Currently when building gather/buildvector node, we try to build nodes shuffles without taking into account separate vector registers. We can improve final codegen and the whole vectorization process by including this info into the analysis and the vector code emission, allows to emit better vectorized code. Differential Revision: https://reviews.llvm.org/D149742
-
LLVM GN Syncbot authored
-
Alexey Bataev authored
This reverts commit 560bad01 to fix a bug reported in https://lab.llvm.org/buildbot/#/builders/5/builds/37763.
-
Joseph Huber authored
Summary: For the other tests we pass `-nogpulib` to ensure that we set up the needed libraries correctly. However, this caused problems for the non-LTO build and test of Nvidia systems. In general this is because we would do a separate compile of the libomptarget device runtime and then link in that cubin. This exercised the runtime in a lot of ways it's not used to, since doing things this way was hardly expected or tested. This patch disables it only for the Nvidia non-LTO build so that we still get the effect of `--liboimptarget-nvptx-bc-path` rather than ignoring it.
-
Louis Dionne authored
[libc++][tests] Fix a few remaining instances of outdated static assertion regexes in our test suite
-
Igor Zhukov authored
Fixes https://github.com/llvm/llvm-project/issues/61202 Differential Revision: https://reviews.llvm.org/D145376
-
Nikolas Klauser authored
This is an implementation detail for `move_only_function` (and potentially other type-erasing classes). Reviewed By: #libc, ldionne Spies: Mordante, ldionne, EricWF, libcxx-commits Differential Revision: https://reviews.llvm.org/D140259
-
Vlad Serebrennikov authored
This patch refactor said enums to hold total size of a bit-field, and not just non-inherited bits. This brings `Type` and `DeclContext` in line with `Comment` and `Stmt`. It also makes it unnecessary to list all transitive bases of a bit-field as unnamed bit-fields, which makes it more friendly towards debuggers.
-
Egor Zhdan authored
``` clang/include/clang/Lex/Preprocessor.h:2893:14: warning: parameter 'isEnter:' not found in the function declaration [-Wdocumentation] /// \param isEnter: true if this PP is entering a region; otherwise, this PP ^~~~~~~~ clang/include/clang/Lex/Preprocessor.h:2895:14: warning: parameter 'Loc:' not found in the function declaration [-Wdocumentation] /// \param Loc: the location of the entry or exit of a ^~~~ clang/include/clang/Lex/Preprocessor.h:2907:14: warning: parameter 'StartLoc:' not found in the function declaration [-Wdocumentation] /// \param StartLoc: output argument. It will be set to the start location of ^~~~~~~~~ ``` -
hassnaaHamdi authored
This patch adds the feature flag FP8 and the assembly/disassembly for the following instructions of NEON, SVE2 and SME2: * NEON Instructions: + Advanced SIMD two-register miscellaneous: - F1CVTL, F1CVTL2, F2CVTL, F2CVTL2 - BF1CVTL, BF1CVTL2, BF2CVTL, BF2CVTL2 + Advanced SIMD three-register extension: - FCVTN, FCVTN2 (FP32 to FP8) - FCVTN (FP16 to FP8) + Advanced SIMD three same: - FSCALE * SVE2 Instructions: + Downconvert instructions: - FCVTN_Z2Z_HtoB - FCVTNB_Z2Z_StoB - BFCVTN_Z2Z_HtoB - FCVTNT_Z2Z_StoB + Upconvert instructions: - F1CVT_ZZ, F2CVT_ZZ - BF1CVT_ZZ, BF2CVT_ZZ - F1CVTLT_ZZ, F2CVTLT_ZZ - BF1CVTLT_ZZ, BF2CVTLT_ZZ * SME2 Instructions: - F1CVT_2ZZ, F2CVT_2ZZ - BF1CVT_2ZZ, BF2CVT_2ZZ - F1CVTL_2ZZ, F2CVTL_2ZZ - BF1CVTL_2ZZ, BF2CVTL_2ZZ - FCVT_Z2Z_HtoB, BFCVT_Z2Z_HtoB - FCVT_Z4Z - FCVTN_Z4Z - FSCALE_2ZZ, FSCALE_4ZZ - FSCALE_2Z2Z, FSCALE_4Z4Z That is according to this documentation: https://developer.arm.com/documentation/ddi0602/2023-09 -
Craig Topper authored
-
Craig Topper authored
I believe this assert was trying to check that 3 variables were equal to 0. I think it instead got interpreted as ((DSWCount == DSWWithPermCount) == DSWWithSharedVMEMCount) == 0 I guess (DSWCount == DSWWithPermCount) was true because both counts were 0. Then true got compared to DSWWithSharedVMEMCount, and since DSWWithSharedVMEMCount is 0, that compare was false. And then that false compared equal to the final 0.
-
Jay Foad authored
-
Jay Foad authored
This helped to catch a codegen failure caused by #69703. MachineVerifier did not complain about this malformed COPY either before regalloc: %9:vreg_64 = COPY %17:vgpr_32 Or after regalloc: renamable $vgpr0_vgpr1 = COPY renamable $vgpr2, implicit $exec But we can at least catch the problem when copyPhysReg tries to expand it into 32-bit register moves and fails to find suitable source registers: $vgpr0 = V_MOV_B32_e32 $noreg, implicit $exec, implicit-def $vgpr0_vgpr1, implicit $vgpr2 $vgpr1 = V_MOV_B32_e32 $noreg, implicit $exec, implicit $vgpr2, implicit $exec
-
Kristof Beyls authored
Also drop phabricator names as we no longer use phabricator. --------- Co-authored-by:Andy Kaylor <andrew.kaylor@intel.com>
-
Nico Weber authored
Getting lots of `error: unknown type name 'uint64_t'` and `uint32_t` in this file on Linux.
-