- Sep 15, 2020
-
-
Guozhi Wei authored
The condition used to decide if need to copy probability should be reversed. Differential Revision: https://reviews.llvm.org/D87417
-
Mark de Wever authored
Before tackling http://llvm.org/PR38722, make sure there is a baseline benchmark. Differential Revision: https://reviews.llvm.org/D62778
-
jasonliu authored
This is a follow up commit for the issue raised in https://reviews.llvm.org/D86879
-
Kristóf Umann authored
The summary and very short discussion in D82122 summarizes whats happening here. In short, liveness talks about variables, or expressions, anything that has a value. Well, statements just simply don't have a one. Differential Revision: https://reviews.llvm.org/D82598
-
Valentin Clement authored
Add a verifier for the loop op in the OpenACC dialect. Check basic restriction from 2.9 Loop construct from the OpenACC 3.0 specs. Reviewed By: ftynse Differential Revision: https://reviews.llvm.org/D87546
-
Simon Pilgrim authored
Forward declare AAResults instead of the (old) AliasAnalysis type. Remove includes from SLPVectorizer.cpp that are already included in SLPVectorizer.h.
-
Marshall Clow authored
[libc++] Use allocator_traits to consistently allocate/deallocate/construct/destroy objects in std::any https://llvm.org/PR45099 notes (correctly) that we're inconsistent in memory allocation in `std::any`. We allocate memory with `std::allocator<T>::allocate`, construct with placement new, destroy by calling the destructor directly, and deallocate by calling `delete`. Most of those are customizable by the user, but in different ways. The standard is silent on how these things are to be accomplished. This patch makes it so we use `allocator_traits<allocator<T>>` for all of these operations (allocate, construct, destruct, deallocate). This is, at least, consistent. Fixes https://llvm.org/PR45099. Differential Revision: https://reviews.llvm.org/D81133
-
Sanjay Patel authored
If the constant operand is the opposite of the min/max value, then the result must be the other value. This is based on the similar codegen transform proposed in: D87571
-
Kristóf Umann authored
Differential Revision: https://reviews.llvm.org/D86532
-
Georgii Rymar authored
[llvm-readobj/elf] - Don't crash when the size of s dynamic symbol table, inferred from the hash table, is broken. Currently we might derive the dynamic symbol table size from the DT_HASH hash table (using its `nchain` field). It is possible to crash dumpers with a broken relocation that refers to a symbol with an index that is too large. To trigger it, the inferred size of the dynamic symbol table should go past the end of the object. This patch adds a size validation + warning. Differential revision: https://reviews.llvm.org/D86923
-
Gabor Marton authored
Add the BufferSize argument constraint to fread and fwrite. This change itself makes it possible to discover a security critical case, described in SEI-CERT ARR38-C. We also add the not-null constraint on the 3rd arguments. In this patch, I also remove those lambdas that don't take any parameters (Fwrite, Fread, Getc), thus making the code better structured. Differential Revision: https://reviews.llvm.org/D87081
-
Guillaume Chatelet authored
Differential Revision: https://reviews.llvm.org/D87689
-
Guillaume Chatelet authored
Differential Revision: https://reviews.llvm.org/D87690
-
Guillaume Chatelet authored
Differential Revision: https://reviews.llvm.org/D87687
-
Stefan Pintilie authored
Add Thread Local Storage Local Exec support to LLD. This is to support PC Relative addressing of Local Exec. The patch teaches LLD to handle: ``` paddi r9, r13, x1@tprel ``` The relocation is: ``` R_PPC_TPREL34 ``` Reviewed By: NeHuang, MaskRay Differential Revision: https://reviews.llvm.org/D86608
-
Qiu Chaofan authored
2508ef01 doesn't totally fix the issue since we did not handle the case when unused temporary negated result is the same with the result, which is found by address sanitizer.
-
Simon Pilgrim authored
GetElementPtrInst::Create returns a GetElementPtrInst* so we don't need to cast. Similarly IntegerType inherits from the Type base class. Also, I've used auto* in a few places to cleanup the code. Helps fix some clang-tidy warnings which saw the dyn_casts and warned that these can return null.
-
Simon Pilgrim authored
The FailureReason input parameter maybe null, we check this in all other cases in the method but this one was missed somehow. Fixes clang-tidy warning.
-
Valentin Clement authored
This patch adds the missing print for the vector_length in the parallel operation. Reviewed By: ftynse Differential Revision: https://reviews.llvm.org/D87630
-
Stephan Herhut authored
This add canonicalizer for - extracting an element from a dynamic_tensor_from_elements - propagating constant operands to the type of dynamic_tensor_from_elements Differential Revision: https://reviews.llvm.org/D87525
-
LLVM GN Syncbot authored
-
LLVM GN Syncbot authored
-
Sanjay Patel authored
There at least one other bug related to pow -> sqrt transforms: http://lists.llvm.org/pipermail/llvm-dev/2020-September/145051.html ...but we probably can't solve that without fixing this first.
-
Sanjay Patel authored
The code drops the sqrt op instead of bailing out, so this is very wrong.
-
Sanjay Patel authored
This is not a valid transform unless we can prove that the program does not read errno after the pow call and before some other function changes it.
-
Oliver Stannard authored
This test tries to create a 2 GiB std::string, catching the bad_alloc exception if the allocation fails. However, for no-exceptions builds there is no way for the error to be reported, so this crashes with a null pointer dereference. Differential revision: https://reviews.llvm.org/D87682
-
Florian Hahn authored
-
Nico Weber authored
-
Sam Clegg authored
With https://reviews.llvm.org/D87537 we made it an error to import or export a mutable global with the +mutable-globals feature present. However the scan was of the entire symbol table rather than just the imports or exports and the filter didn't match exaclyt meaning the `__stack_pointer` (a mutable global) was always triggering with error when the `--export-all` flag was used. This also revealed that we didn't have any test coverage for the `--export-all` flag. This change fixes the current breakage on the emscripten-releases roller. Differential Revision: https://reviews.llvm.org/D87663
-
sameeran joshi authored
Adds a new GettingInvolved page to documentation which provides details about mailing list, chats and calls. Adds a sidebar page which provides common links on all documentation pages. The links include: - Getting Started - Getting Involved - Github Repository - Bug Reports - Code Review Depends on https://reviews.llvm.org/D87242 Reviewed By: richard.barton.arm Differential Revision: https://reviews.llvm.org/D87270
-
Florian Hahn authored
This patch adds a isConditionImplied function that takes a constraint and returns true if the constraint is implied by the current constraints in the system. Reviewed By: spatel Differential Revision: https://reviews.llvm.org/D84545
-
Felix Berger authored
Restrict UnnecessaryCopyInitialization check to variables initialized from free functions without arguments This restriction avoids cases where an alias is returned to an argument and which could lead to to a false positive change.
-
Louis Dionne authored
Setting _LIBCPP_HAS_NO_THREADS is needed when building libcxxabi without threads in standalone mode. This is useful when target WASM. Otherwise, you get an error like "No thread API" when building libcxxabi. It would be better to link against a properly-configured libc++ headers CMake target when building libc++abi instead, but we don't generate such targets yet. Thanks to Matthew Bauer for the patch. Differential Revision: https://reviews.llvm.org/D60743
-
Simon Pilgrim authored
More remaining dependencies down to LoopCacheAnalysis.cpp
-
Simon Pilgrim authored
Forward declare AAResults instead of the (old) AliasAnalysis type.
-
Simon Pilgrim authored
These are all directly included in AliasSetTracker.h
-
Sjoerd Meijer authored
Loop tripcount expressions have a positive range, so use unsigned SCEV ranges for them. Differential Revision: https://reviews.llvm.org/D87608
-
Bjorn Pettersson authored
The "takeName" logic in ScalarizerVisitor::gather did not consider that the value vector could refer to non-instructions, such as global variables. This patch make sure that we avoid changing the name of a value if it isn't an instruction. Reviewed By: lebedev.ri Differential Revision: https://reviews.llvm.org/D87685
-
Pavel Labath authored
qemu calls the "fp" and "lr" registers via their generic names (x29/x30). This mismatch manifested itself as not being able to unwind or display values of some local variables.
-
Pavel Labath authored
lldbUtilityHelpers does not depend on lldbSymbolHelpers. Remove that dependency, and add direct lldbSymbolHelpers dependencies where needed.
-