- Nov 23, 2022
-
-
Hassnaa Hamdi authored
1-To generate code compatible to streaming mode: - enable custom lowering for SMIN, SMAX, UMIN, UMAX, FMAXNUM, FMINNUM, FMAXIMUM, FMINIMUM. 2-Testing files: - int-minmax.ll - fp-minmax.ll Reviewed By: david-arm, sdesmalen Differential Revision: https://reviews.llvm.org/D138292
-
Hassnaa Hamdi authored
Add testing files: - int-minmax.ll - fp-minmax.ll
-
Louis Dionne authored
-
Hans Wennborg authored
It broke the build, see comments on code review. > Leaves the implementation and tests files in-place for right now, but > deletes the ability to build the old sanitizer-common based scudo. This > has been on life-support for a long time, and the newer scudo_standalone > is much better supported and maintained. > > Also patches up some GWP-ASan wording, primarily related to the fact > that -fsanitize=scudo now is scudo_standalone, and therefore the way to > reference the GWP-ASan options through the environment variable has > changed. > > Future follow-up patches will delete the original scudo, and migrate all > its tests over to be part of the scudo_standalone test suite. > > Reviewed By: vitalybuka > > Differential Revision: https://reviews.llvm.org/D138157 This reverts commit ab1a5991.
-
Jonas Paulsson authored
In cases where the SELECT_CCMASK has an additional user of the carry, a truncated SELECT_CCMASK may result as the input to the GET_CCMASK, which need to be recognized. Fixes https://github.com/llvm/llvm-project/issues/59054 Reviewed By: Ulrich Weigand Differential Revision: https://reviews.llvm.org/D138324
-
Balazs Benics authored
Casting a pointer to a suitably large integral type by reinterpret-cast should result in the same value as by using the `__builtin_bit_cast()`. The compiler exploits this: https://godbolt.org/z/zMP3sG683 However, the analyzer does not bind the same symbolic value to these expressions, resulting in weird situations, such as failing equality checks and even results in crashes: https://godbolt.org/z/oeMP7cj8q Previously, in the `RegionStoreManager::getBinding()` even if `T` was non-null, we replaced it with `TVR->getValueType()` in case the `MR` was `TypedValueRegion`. It doesn't make much sense to auto-detect the type if the type is already given. By not doing the auto-detection, we would just do the right thing and perform the load by that type. This means that we will cast the value to that type. So, in this patch, I'm proposing to do auto-detection only if the type was null. Here is a snippet of code, annotated by the previous and new dump values. `LocAsInteger` should wrap the `SymRegion`, since we want to load the address as if it was an integer. In none of the following cases should type auto-detection be triggered, hence we should eventually reach an `evalCast()` to lazily cast the loaded value into that type. ```lang=C++ void LValueToRValueBitCast_dumps(void *p, char (*array)[8]) { clang_analyzer_dump(p); // remained: &SymRegion{reg_$0<void * p>} clang_analyzer_dump(array); // remained: {{&SymRegion{reg_$1<char (*)[8] array>} clang_analyzer_dump((unsigned long)p); // remained: {{&SymRegion{reg_$0<void * p>} [as 64 bit integer]}} clang_analyzer_dump(__builtin_bit_cast(unsigned long, p)); <--------- change #1 // previously: {{&SymRegion{reg_$0<void * p>}}} // now: {{&SymRegion{reg_$0<void * p>} [as 64 bit integer]}} clang_analyzer_dump((unsigned long)array); // remained: {{&SymRegion{reg_$1<char (*)[8] array>} [as 64 bit integer]}} clang_analyzer_dump(__builtin_bit_cast(unsigned long, array)); <--------- change #2 // previously: {{&SymRegion{reg_$1<char (*)[8] array>}}} // now: {{&SymRegion{reg_$1<char (*)[8] array>} [as 64 bit integer]}} } ``` Reviewed By: xazax.hun Differential Revision: https://reviews.llvm.org/D136603
-
Louis Dionne authored
This patch removes the base template implementation for std::char_traits. If my reading of http://eel.is/c++draft/char.traits is correct, the Standard mandates that the library provides specializations for several types like char and wchar_t, but not any implementation in the base template. Indeed, such an implementation is bound to be incorrect for most types anyways, since things like `eof()` and `int_type` will definitely have to be customized. Since the base template implementation should not have worked for anyone, this shouldn't be a breaking change (I expect that anyone defining a custom character type today will already have to provide their own specialization of char_traits). However, since we're aware of some users of char_traits for unsigned char and signed char, we're keeping those two specializations around for two releases to give people some time to migrate. Differential Revision: https://reviews.llvm.org/D138307
-
Louis Dionne authored
This makes the library consistent in how it handles C library headers. For C headers provided by libc++, we unconditionally include <foo.h> from <cfoo>, and then <foo.h> conditionally include_next <foo.h>. For headers not provided by libc++, <cfoo> conditionally includes the system's <foo.h> directly. Differential Revision: https://reviews.llvm.org/D138512
-
Nico Weber authored
No behavior difference in practice, but makes it possible to use `-t` for debugging when that assert fails.
-
Michał Górny authored
Update external unittest rules for flang standalone builds to account for the unittest components (gtest, gmock) being moved to `third-party` directory, in a11cd0d9. Differential Revision: https://reviews.llvm.org/D138562
-
Emmmer authored
Add: - RV32F instruction set. - corresponding unittests. Further work: - RV32FC, RV64F and RV64FC instructions support. - update execution exceptions to fcsr register in RVM instructions. Reviewed By: DavidSpickett Differential Revision: https://reviews.llvm.org/D138447
-
OCHyams authored
Fix replaceVariableLocationOp unconditionally replacing the first operand of a dbg.assign. Reviewed By: jryans Differential Revision: https://reviews.llvm.org/D138561
-
Matt Arsenault authored
-
Louis Dionne authored
-
Matt Devereau authored
This reverts a change to exclude scalarizeBinopOrCmp in VectorCombine for scalable vectors which caused poor scalable Binop codegen. Differential Revision: https://reviews.llvm.org/D138545
-
Matthias Springer authored
Do not generate CollapseShapeOps/ExpandShapeOps that have the same source and result shape. Generate casts instead. Such reshapes became invalid with D138498. Differential Revision: https://reviews.llvm.org/D138557
-
Pavel Labath authored
This reverts commit f0c16f89 because it breaks linux and mac bots.
-
Benjamin Kramer authored
This reverts commit cf624b23. It triggers crashes in clang, see the comments on github on the original change.
-
David Green authored
I noticed these were missing, so this adds Host identifiers for cortex-a55, cortex-a510, cortex-a710 and cortex-x2, taken from their respective TRMs. Differential Revision: https://reviews.llvm.org/D138497
-
zhongyunde authored
* For case bcmp9, there is extras AND and EXTEND int the chain of OR/XOR, which prevent the transform, so enable the optimize before legalization. * The key IR frag related: t37: i32,ch = load<(load (s8) from %ir.4), anyext from i8> t0, t11, undef:i64 t12: i64 = add t4, Constant:i64<8> t38: i32,ch = load<(load (s8) from %ir.5), anyext from i8> t0, t12, undef:i64 t39: i32 = xor t37, t38 t40: i64 = any_extend t39 t42: i64 = and t40, Constant:i64<255> Depends on D138398 to fix combine_setcc_glue Reviewed By: dmgreen, bcl5980 Differential Revision: https://reviews.llvm.org/D137936 -
Haohai Wen authored
Avx512 permute latency should be 3 instead of 1. Reviewed By: RKSimon Differential Revision: https://reviews.llvm.org/D138427
-
Stefan Gränitz authored
This reverts commit a37807ac. Differential Revision: https://reviews.llvm.org/D137942
-
Stefan Gränitz authored
When ObjCARCOpt::run() returned early, Changed and CFGChanged were never initialized. CFGChanged is read unconditionally afterwards. This came up in the course of D137942.
-
Benjamin Kramer authored
-
Alex Richardson authored
The ctype mask for newlib/picolibc is fully saturated, so __regex_word has to overlap with one of the values. This commit uses the same workaround as bionic did (uint16_t for char_class_type inside regex_traits). It should be possible to have libc++ provide the default rune table instead, but that will require a new mechanism to detect newlib inside __config since the header defining the newlib/picolibc macros has not been included yet inside __config. Doing it this way also avoids duplicating the ctype table for newlib, reducing the global data size. Differential Revision: https://reviews.llvm.org/D138195
-
Alex Richardson authored
I noticed a an assertion error when building MIPS code that loaded from NULL. Loading from NULL ends up being a load with maximum alignment, and due to integer truncation the value maximum was interpreted as 0 and the assertion in MipsDAGToDAGISel::Select() failed. This previously happened to work, but the maximum alignment was increased in df84c1fe, so it no longer fits into a 32 bit integer. Instead of just fixing the one MIPS case, this patch removes all uses of the deprecated getAlignment() call and replaces them with getAlign(). Differential Revision: https://reviews.llvm.org/D138420
-
Alex Richardson authored
This simply calls getTargetAddressSpace(Q.getAddressSpace()) and there are only two callers, so adjust those caller instead.
-
Kito Cheng authored
Introduce a RVVTypeCache to hold the cache instead of using a local static variable to maintain a cache. Also made construct of RVVType to private, make sure that could be only created by a cache manager. Reviewed By: sammccall Differential Revision: https://reviews.llvm.org/D138429
-
Akshay Baviskar authored
Adds RegionBranchOpInterface for AffineIf Op and tests it using buffer deallocation pass. Reviewed By: bondhugula Differential Revision: https://reviews.llvm.org/D130962
-
Matthias Springer authored
CollapseShapeOp/ExpandShapeOp that do not change the rank (or increase/reduce it) are invalid. Differential Revision: https://reviews.llvm.org/D138498
-
Nicolas Vasilache authored
This revision refactors and cleans up a bunch of infra related to vector, shapes and indexing into more reusable APIs. Differential Revision: https://reviews.llvm.org/D138501
-
Max Kazantsev authored
-
WuXinlong authored
This patch add the instructions of Zcf extension. Zcf is a subset of C Ext which include the single-precision floating-point instructions. Reviewed By: craig.topper Differential Revision: https://reviews.llvm.org/D134176
-
Tom Stellard authored
Reviewed By: mgorny Differential Revision: https://reviews.llvm.org/D137854
-
Michał Górny authored
Fix Findzstd CMake to handle shared libraries on OpenBSD correctly. This userland does not use shared library symlinks without SOVERSION, so the result of find_library() does not ever end with zstd_SHARED_LIBRARY_SUFFIX. To work around this, reverse the logic to compare the result against zstd_STATIC_LIBRARY_SUFFIX and assume shared library otherwise. While at it, fix the conditions not to fall back to "result is static library" path if it actually was recognized as a shared library but zstd_shared target already existed. Fixes #59056 Differential Revision: https://reviews.llvm.org/D138361
-
Fangrui Song authored
We can add .riscv.attributes synthetic section here in the future.
-
Philip Reames authored
Most of the code for this was taken from https://reviews.llvm.org/D133552, with one bug fix by me. I'm landing the plumbing so that we can focus on the cost model pieces in the review.
-
Daniel Rodríguez Troitiño authored
The exports trie used to be pointed by the information in LC_DYLD_INFO, but when chained fixups are present, the exports trie is pointed by LC_DYLD_EXPORTS_TRIE instead. Modify ObjCopy code to calculate the right offset and size needed depending on the existence of LC_DYLD_INFO or LC_DYLD_EXPORTS_TRIE, read the exports from either of those places, and write the export information as pointed to either of those places. Depends on D134571. Reviewed By: alexander-shaposhnikov Differential Revision: https://reviews.llvm.org/D137879
-
Matt Arsenault authored
-