- Feb 10, 2020
-
-
Eric Christopher authored
-
https://reviews.llvm.org/D74137Max Moroz authored
Summary: The refactoring has caused a failure in http://lab.llvm.org:8011/builders/clang-ppc64le-linux/builds/29265 The idea of failing the symbolization when the symbolizer bufer is too small was incorrect. The symbolizer can be invoked for other frames that may fit into the buffer and get symbolized. Reviewers: vitalybuka, eugenis Subscribers: dberris, #sanitizers, llvm-commits Tags: #sanitizers, #llvm Differential Revision: https://reviews.llvm.org/D74343
-
Florian Hahn authored
ConstantInt values are always represented as constant ranges with a single element. getConstantInt is obsolete, as pointed out by @nikic during D60581. Reviewers: nikic Reviewed By: nikic Differential Revision: https://reviews.llvm.org/D74329
-
Stephen Neuendorffer authored
-
Stephen Neuendorffer authored
-
Stephen Neuendorffer authored
Differential Revision: https://reviews.llvm.org/D73672
-
Stephen Neuendorffer authored
Allow this only if the types are the same. e.g.: i16 -> index -> i16 or index -> i16 -> index Differential Revision: https://reviews.llvm.org/D73671
-
Stephen Neuendorffer authored
NFC for binary operations. Differential Revision: https://reviews.llvm.org/D73670
-
Rachel Craik authored
LoopCacheAnalysis currently assumes the loop will be iterated over in a forward direction. This patch addresses the issue by using the absolute value of the stride when iterating backwards. Note: this patch will treat negative and positive array access the same, resulting in the same cost being calculated for single and bi-directional access patterns. This should be improved in a subsequent patch. Reviewed By: jdoerfert Differential Revision: https://reviews.llvm.org/D73064
-
Dimitry Andric authored
Summary: Instead of hand-crafting an offset into the structure returned by dlopen(3) to get at the link map, use the documented API. This is described in dlinfo(3): by calling it with `RTLD_DI_LINKMAP`, the dynamic linker ensures the right address is returned. Reviewers: devnexen, emaste, MaskRay, krytarowski Reviewed By: krytarowski Subscribers: krytarowski, vitalybuka, #sanitizers, llvm-commits Tags: #sanitizers, #llvm Differential Revision: https://reviews.llvm.org/D73990
-
Vedant Kumar authored
Look up the -arch flags to pass to the mig invocation from an optionally-defined MIG_ARCHS variable. We can't use CMAKE_OSX_ARCHS because the {i,tv,watch}OS builds don't use this mechanism to achieve fat builds (they build each slice separately & then lipo them together). This supercedes the mig -arch/-isysroot fix from 510758da.
-
Jonas Paulsson authored
Each function is with this compiled with the SystemZSubtarget initialized from the functions attributes. Review: Ulrich Weigand. Differential Revision: https://reviews.llvm.org/D74086
-
Eric Christopher authored
http://lists.llvm.org/pipermail/llvm-dev/2020-February/139058.html Approved by dblaikie, pcc.
-
Vedant Kumar authored
Differential Revision: https://reviews.llvm.org/D73938
-
Matt Arsenault authored
Instead of handling the r600 intrinsics on amdgcn, handle the amdgcn intrinsics on r600.
-
Simon Pilgrim authored
-
Simon Pilgrim authored
-
Michael Kruse authored
The idiom for (auto i = n - n; i < n; i += 1) was intended to automatically derive the type of i from n (signed/unsigned int) and avoid the 'mixed signed/unsigned comparison' warning. However, almost-always-auto was never used in the LLVM coding style (although we used it in Polly for some time) and I did never intended to use this idiom upstream. PVS Studio may warns about this idiom as 'warning: both sides of operator are equivalent [misc-redundant-expression]'. Remove the use of auto and directly use unsigned. Also see http://llvm.org/PR44768
-
aartbik authored
Summary: This allows for lowering of VectorOps (and others) into a LLVM IR that maps directly to efficient implementations on the target machines. http://llvm.org/docs/LangRef.html#experimental-vector-reduction-intrinsics Reviewers: ftynse, andydavis1, nicolasvasilache, rriddle Reviewed By: ftynse, rriddle Subscribers: jfb, mehdi_amini, rriddle, jpienaar, burmako, shauheen, antiagainst, nicolasvasilache, arpith-jacob, mgester, lucyrfox, liufengdb, Joonsoo, llvm-commits Tags: #llvm Differential Revision: https://reviews.llvm.org/D74171
-
Nicolas Vasilache authored
This CL refactors EDSCs to layer them better and break unnecessary dependencies. After this refactoring, the top-level EDSC target only depends on IR but not on Dialects anymore and each dialect has its own EDSC directory. This simplifies the layering and breaks cyclic dependencies. In particular, the declarative builder + folder are made explicit and are now confined to Linalg. As the refactoring occurred, certain classes and abstractions that were not paying for themselves have been removed. Differential Revision: https://reviews.llvm.org/D74302
-
Simon Pilgrim authored
Non-AVX512BW targets failed to concatenate 256-bit shifts back to 512-bits (split during 512-bit shuffle lowering as they don't have v32i16/v64i8 types).
-
David Stenberg authored
This reverts commit b54a8ec1. The commit triggered debug invariance (different output with/without -g). The patch seems to have exposed a pre-existing invariance problem in GlobalOpt, which I'll write a bug report for.
-
Kadir Cetinkaya authored
-
Saleem Abdulrasool authored
This patch renames `__personality_routine` to `_Unwind_Personality_Fn` in `unwind.h`. Both `unwind.h` from clang and GCC headers use this name instead of `__personality_routine`. With this patch one is also able to build libc++abi with libunwind support on Windows. Patch by Markus Böck!
-
Tobias Gysi authored
The current standard to llvm conversion pass lowers subview ops only if dynamic offsets are provided. This commit extends the lowering with a code path that uses the constant offset of the target memref for the subview op lowering (see Example 3 of the subview op definition for an example) if no dynamic offsets are provided. Differential Revision: https://reviews.llvm.org/D74280
-
Stanislav Mekhanoshin authored
These are generated and do not need to have the same values. We are defining separate subregs for R600 and GCN but then using AMDGPU subregs on R600. Differential Revision: https://reviews.llvm.org/D74248
-
James Henderson authored
The test previously assumed that the host was little endian, which broke the big endian build bots.
-
Simon Pilgrim authored
As noted on PR44379, we didn't attempt to lower vector shuffles using bit rotations on XOP/AVX512F targets. This patch lowers to uniform ISD:ROTL nodes - ROTR isn't supported by XOP and they are interchangeable for constant values anyway. There might be cases where targets without ISD:ROTL support would benefit from this (expanding to SRL+SHL+OR), which I'll investigate in a future patch. REAPPLIED rGe82e17d4d4ca after reversion at rG39eade73a567 - fixed offset matching in matchShuffleAsBitRotate.
-
LLVM GN Syncbot authored
-
Michael Wyman authored
Summary: Such implementations may override the class's own implementation, and even be a danger in case someone later comes and adds one to the class itself. Most times this has been encountered have been a mistake. Reviewers: stephanemoore, benhamilton, dmaclach Reviewed By: stephanemoore, benhamilton, dmaclach Subscribers: dmaclach, mgorny, cfe-commits Tags: #clang-tools-extra, #clang Differential Revision: https://reviews.llvm.org/D72876
-
Gabor Marton authored
Reviewers: NoQ Tags: #clang Differential Revision: https://reviews.llvm.org/D73897
-
Kadir Cetinkaya authored
This breaks under asan, see http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux-fast/builds/38597/steps/check-clang%20asan/logs/stdio This reverts commit bb504542. Revert "[FIX] Ordering problem accidentally introduced with D72304" This reverts commit 08c0a06d. Revert "[OpenMP][OMPIRBuilder] Add Directives (master and critical) to OMPBuilder." This reverts commit e8a436c5.
-
Florian Hahn authored
-memdep-block-scan-limit is not relevant with MSSA.
-
James Henderson authored
-
Bill Wendling authored
The build failed with error: call to deleted constructor of 'llvm::Error' errors. This reverts commit 1c2241a7.
-
Max Moroz authored
Summary: Nothing critical, just a few potential improvements I've noticed while reading the code: - return `false` when symbolizer buffer is too small to read all data - invert some conditions to reduce indentation - prefer `nullptr` over `0` for pointers; init some pointers on stack; - remove minor code duplication Reviewers: eugenis, vitalybuka Subscribers: dberris, #sanitizers, llvm-commits, kcc Tags: #sanitizers, #llvm Differential Revision: https://reviews.llvm.org/D74137
-
James Henderson authored
If a debug line section with version of greater than 5 is encountered, prior to this change the parser would accept it and treat it as version 5. This might work to some extent, but then it might not at all, as it really depends on the format of the unspecified future version, which will be different (otherwise there would be no point in changing the version number). Any information we could provide has a good chance of being invalid, so we should just refuse to parse such tables. Reviewed by: dblaikie, MaskRay Differential Revision: https://reviews.llvm.org/D74204
-
James Henderson authored
-
Bill Wendling authored
-
Luke Geeson authored
This patch makes the following System Registers Read Only: - CurrentEL - ICH_MISR_EL2 - PMBIDR_EL1 - PMSIDR_EL1 as found in: https://developer.arm.com/docs/ddi0595/e/aarch64-system-registers Relative line numbers were also added to the tests so we get more informative error messages on failure. Change-Id: I963b4f01ca5737b58f9e8e7abe9ca1d99e328758
-