- Mar 08, 2023
-
-
Paul Walker authored
I've pulled this change from D145404 to land in isolation because I'm concerned the code might be more important than the test coverage might suggest (NOTE: the code has no test coverage).
-
Nikolas Klauser authored
This reverts commit a9356a51.
-
Michael Buch authored
-
Xi Ruoyao authored
Without the definition, build fails on AArch64 with > error: 'AT_HWCAP2' undeclared (first use in this function); > did you mean 'AT_HWCAP'? with old Glibc versions. Differential Revision: https://reviews.llvm.org/D145494
-
Ilya Leoshkevich authored
Glibc provides the argp_parse() function for parsing command line arguments [1]. Indicate that argc/argv are read from and arg_index is written to. Strictly speaking, we also need to indicate that argp is read from, but this would require describing its layout, and most people use a static initializer there, so it's not worth the effort. [1] https://www.gnu.org/software/libc/manual/html_node/Argp.html Reviewed By: vitalybuka Differential Revision: https://reviews.llvm.org/D143330
-
Simon Pilgrim authored
-
Alex Richardson authored
When building compiler-rt builtins for x86_64 they library will by default also be built for i386. We unconditionally add the Float16 compile flags since the check for Float16 support will be done using x86_64 compiler flags, but i386 does not actually support it. Fix this by moving the COMPILER_RT_HAS_FLOAT16 and COMPILER_RT_HAS_FLOAT16 checks to a per-target-architecture check inside the loop (using `check_c_source_compiles` and `cmake_{push,pop}_check_state`). Many of the checks in the builtin-config-ix file should probably also be changed to per-target-arch checks, but so far only the Float16 one has caused issues. This is an alternative to D136044 which added a special case for i386 FreeBSD. Fixes: https://github.com/llvm/llvm-project/issues/57224 Differential Revision: https://reviews.llvm.org/D145237
-
David Spickett authored
The last user was ProcessMessage, which has itself been removed. Reviewed By: emaste Differential Revision: https://reviews.llvm.org/D145561
-
Nikolas Klauser authored
This check flags code which uses `_LIBCPP_STD_VER` the wrong way, or tries to use `__cplusplus`. It flags cases where we use `_LIBCPP_STD_VER >` instead of `_LIBCPP_STD_VER >=` and where wrong values are used (e.g. `_LIBCPP_STD_VER >= 24`). Reviewed By: ldionne, #libc Spies: libcxx-commits Differential Revision: https://reviews.llvm.org/D144261
-
Shivam Gupta authored
Differential Revision: https://reviews.llvm.org/D145554
-
David Truby authored
This implements the atand intrinsic by performing a multiplication by 180/pi to the result of a call to atan inline. Differential Revision: https://reviews.llvm.org/D144885
-
Siva Chandra authored
Reviewed By: mikhail.ramalho Differential Revision: https://reviews.llvm.org/D145560
-
Viktoriia Bakalova authored
This reverts commit fd8c9ef2. Differential Revision: https://reviews.llvm.org/D145577
-
Guillaume Chatelet authored
In the same vein as https://reviews.llvm.org/D141553 Enable the feature globally to ensure layering and catch circular dependencies (https://llvm.org/docs/CodingStandards.html#library-layering). Differential Revision: https://reviews.llvm.org/D143678
-
David Spickett authored
The last use of these was removed in cd443398. Reviewed By: emaste Differential Revision: https://reviews.llvm.org/D145559
-
Valentin Clement authored
-
David Green authored
These should have been using the LDURBi instructions where the offset is negative, as reported from the reproducer in D144086.
-
Viktoriia Bakalova authored
This reverts commit 85a5c17b.
-
Viktoriia Bakalova authored
This reverts commit 7bd56dda.
-
Viktoriia Bakalova authored
This reverts commit 9814b4d0.
-
Viktoriia Bakalova authored
This reverts commit 85a5c17b.
-
Viktoriia Bakalova authored
This reverts commit 2eb5ac99. Differential Revision: https://reviews.llvm.org/D145576
-
Mikhail R. Gadelha authored
This patch includes: 1. Better error message when cmake finds incompatible triple 2. Added missing header dependencies libc/include/CMakeLists.txt as per app.td 3. Removed unused $LLVM_LIBC_INCLUDE_DIRS cmake variable Reviewed By: sivachandra Differential Revision: https://reviews.llvm.org/D145496
-
Carl Ritson authored
This reverts commit be0ef4b9. Investigating use after free issue.
-
Valery Pykhtin authored
We reuse live registers after tracking one MBB as live-ins to the successor MBB if the successor is only one but we don't check if the successor has other predecessors. `A B` ` \ /` ` C` A and B have one successor but C has live-ins defined by A and B and therefore should be initialized using LIS. This fixes 83 lit tests out if 420 with EXPENSIVE_CHECK enabled. Reviewed By: rampitec Differential Revision: https://reviews.llvm.org/D136918
-
Florian Hahn authored
Update less_first,less_second to use std::get to access the first and second component. This extends support to any type implementing std::get, like tuples. Reviewed By: dblaikie Differential Revision: https://reviews.llvm.org/D145489
-
Viktoriia Bakalova authored
This reverts commit 2eb5ac99.
-
Alex Bradbury authored
Differential Revision: https://reviews.llvm.org/D144465
-
Michael Buch authored
This got fixed in D145241 Differential Revision: https://reviews.llvm.org/D145241
-
Florian Hahn authored
Check if replicate recipe is in a replicate region when considering to collect predicated instructions. This allows use IsPredicated for recipes with a mask attached directly in D143865. Reviewed By: Ayal Differential Revision: https://reviews.llvm.org/D145322
-
Adrian Kuegel authored
- 'override' is redundant since the function is already declared 'final'. - 'virtual' is redundant since the function is already declared 'override'.
-
Simon Pilgrim authored
Reduces the superfluous diffs in D145540
-
Stanislav Mekhanoshin authored
Memory models for gfx90a and gfx940 do not require buffer_wbl2 before the fence for acquire ordering, but we do insert the full release. Fixes: SWDEV-386785 Differential Revision: https://reviews.llvm.org/D145524
-
Valentin Clement authored
Unallocated unlimited polymorphic entities do not have a dynamic type set and do not have declared type. The standard notes that the result is processor dependent when one of the arguments of same_type_as is in this case. Align the result to other compiler (gfortran, nvfortran). Reviewed By: jeanPerier, PeteSteinfeld Differential Revision: https://reviews.llvm.org/D145384
-
Chuanqi Xu authored
Close https://github.com/llvm/llvm-project/issues/60085 When I look into pr60085, I finf the issue gone away now surprisingly. Leave a test for this to prevent it happen again.
-
Andrzej Warzynski authored
Vectorization of `tensor.extract` using contiguous loads (`vector.transfer_read`) was introduced in [1]. This patch updates and refines the existing logic (so that more cases of contiguous can be identified), as well as adds more tests. Specifically, contiguous load operations are identified by making sure that: 1. non-trailing indices for `tensor.extract` are loop invariant (so, e.g., there are no "jumps" from one row to the other between iterations), 2. the trailing index for `tensor.extract` increments by 1 with every loop iteration (so that it's always adjacent elements that are loaded). This patch introduces: * `isLoopInvariantIdx` for step 1., and * `isContiguousLoadIdx` for step 2. These new methods replace: * `isContiguousLoadIdx`, and `isBasedOnIndexOp`. Both approaches lead to similar end-result (none of the existing tests required updating). However, with the updated approach, it's much easier to treat the trailing and non-trailing indices separately and to add more cases for which contiguous loads can be used. [1] https://reviews.llvm.org/D141998 Differential Revision: https://reviews.llvm.org/D145385
-
Yeting Kuo authored
[InstCombine][debuginfo] Update valueCoversEntireFragment for fixed size fragment and scalable value. Update valueCoversEntireFragment to handle new case and add regression test. Reviewed By: sdesmalen, aprantl Differential Revision: https://reviews.llvm.org/D144472
-
Michael Platings authored
The change to potentially use symlinks on Windows was added in https://reviews.llvm.org/D99170. LLVM_USE_SYMLINKS was added more recently in https://reviews.llvm.org/D135578 and allows specifying at configure time whether or not symlinks should be created. The benefit of using this option is it allows building the package on a symlink-capable Windows machine with symlinks disabled so that the resulting package can be used on a Windows machine that doesn't support symlinks. Differential Revision: https://reviews.llvm.org/D145443
-
esmeyi authored
Summary: Fixes #60990. There is a crash reported during Running pass 'Prepare loop for ppc preferred instruction forms'. The crash occurs in 32bit PowerPC. Reviewed By: shchenz Differential Revision: https://reviews.llvm.org/D145350
-
Carl Ritson authored
Ensure metadata for declarations copied during materialization is properly mapped if declarations do not become definitions. Reviewed By: tejohnson Differential Revision: https://reviews.llvm.org/D145318
-