- Feb 17, 2022
-
-
Arthur O'Dwyer authored
Differential Revision: https://reviews.llvm.org/D119863
-
- Feb 16, 2022
-
-
Arthur O'Dwyer authored
Drive-by remove some use of raw `new`, and expand test coverage a tiny bit. Differential Revision: https://reviews.llvm.org/D119861
-
Philip Reames authored
-
Arthur O'Dwyer authored
"cxx2b-consteval-if.cpp" was showing up misindented.
-
Vladislav Khmelevsky authored
This patch changes patchELFAllocatableRelaSections from going through old relocations sections and update the relocation offsets to emitting the relocations stored in binary sections. This is needed in case we would like to remove and add dynamic relocations during BOLT work and it is used by golang support pass. Note: Currently we emit relocations in the old sections, so the total number of them should be equal or less of old number. Testing: No special tests are neeeded, since this patch does not fix anything or add new functionality (it only prepares to add). Every PIC-compiled test binary will use this code and thus become a test. But just in case the aarch64 dynamic relocations tests were added. Vladislav Khmelevsky, Advanced Software Technology Lab, Huawei Reviewed By: maksfb Differential Revision: https://reviews.llvm.org/D117612
-
Nikita Popov authored
By convention, memcpy/memmove intrinsics are always used with i8 pointers (though this is not enforced), so in practice this code was always using an i8 type. Make that explicit. Of course, i8 is not a very profitable choice, and this code could be more performant by picking an appropriate larger type. But that would require additional test coverage and correctness review, and certainly shouldn't be a decision based on the pointer element type.
-
Louis Dionne authored
Otherwise, AppleClang ignores -fmodules entirely, so we are not actually testing anything. Differential Revision: https://reviews.llvm.org/D119862
-
Sven van Haastregt authored
The named and generic address space overloads for atomic_init added by 50f8abb9 ("[OpenCL] Add OpenCL 3.0 atomics to -fdeclare-opencl-builtins", 2022-02-11) were not guarded by the corresponding extensions.
-
Kevin P. Neal authored
Currently the fsub optimizations in InstSimplify don't know how to fold X - -0.0 to X when we know X is not zero and the constrained intrinsics are used. This adds the support. This review is split out from D107285. Differential Revision: https://reviews.llvm.org/D119746
-
Tue Ly authored
Algorithm for hypotf: compute (a*a + b*b) in double precision, then use Dekker's algorithm to find the rounding error, and then correcting it after taking its square-root. Reviewed By: sivachandra Differential Revision: https://reviews.llvm.org/D118157
-
Louis Dionne authored
As suggested in https://reviews.llvm.org/D112155. Differential Revision: https://reviews.llvm.org/D119836
-
Peter Kasting authored
This relands 73e585e4 (and 0574b5fc), with a fix for the failing test (by using Optional<StringRef>s instead of making StringRef::empty() mean absence of value). Differential Revision: https://reviews.llvm.org/D118070
-
Nikita Popov authored
Address space casts in general may change the element type, but don't allow it in the method working on Address, so we can preserve the element type. CreatePointerBitCastOrAddrSpaceCast() still needs to be addressed.
-
Sanjay Patel authored
D119851
-
Florian Hahn authored
Blocks with UnreachableInst terminators are considered as root nodes in the PDT. This pessimize DSE, if there are no aliasing reads from the potentially dead store and the block with the unreachable terminator. If any of the root nodes of the PDF has UnreachableInst as terminator, fall back to the CFG scan, even the common dominator of all killing blocks does not post-dominate the block with potentially dead store. It looks like the compile-time impact for the extra scans is negligible. https://llvm-compile-time-tracker.com/compare.php?from=779bbbf27fe631154bdfaac7a443f198d4654688&to=ac59945f1bec1c6a7d7f5590c8c69fd9c5369c53&stat=instructions Fixes #53800. Reviewed By: nikic Differential Revision: https://reviews.llvm.org/D119760
-
Adrian Kuegel authored
-
Nikita Popov authored
These are leftovers from when there were separate MSSA/non-MSSA check lines.
-
Tim Northover authored
It provides some functions needed by add/sub.
-
Kristof Beyls authored
-
Simon Pilgrim authored
Expand the ISD::AVGCEILU to determine the known bits of the result. First part of PR53622 Differential Revision: https://reviews.llvm.org/D119629
-
Marek Kurdej authored
Fixes https://github.com/llvm/llvm-project/issues/52649. This was already fixed in commit https://github.com/llvm/llvm-project/commit/e967d97a35a93c883528a9672159edff05f5addb.
-
Anastasia Stulova authored
Reflect the latest development in clang docs. Differential Revision: https://reviews.llvm.org/D119719
-
Nathan Sidwell authored
parseNestedName's main loop allowed parsing a grammar that was more flexible than the actual grammar. This refactors that to rule out some more incorrect manglings. 1) The 'L' extension only applies to unqualified-name components, so check it just there. 2) The 'M' suffix is, AFAICT, removed from the grammar. Rather than eliminate it, let's parse it after we've parsed a component. Added some additional bad mangling tests, which are now rejected. I don't break the 'T' and 'D[tT]' cases out of the loop, even though they can only appear at first position, as it seems simpler to just check there is nothing SoFar. Reviewed By: ChuanqiXu Differential Revision: https://reviews.llvm.org/D119542
-
Nathan Sidwell authored
The Itanium mangler constructors use both NSDMI and explicit member construction for default values. This is confusing. *) Use NSDMIs wherever possible *) Use forwarding ctor for the nesting case with an llvm::raw_null_ostream (and explicitly set NullOut flag in that ctor). *) Copy the ModuleSubstitutions. This is a bug with no effect in the current mangling, but not in the newer mangling. Reviewed By: ChuanqiXu Differential Revision: https://reviews.llvm.org/D119550
-
Nemanja Ivanovic authored
The added run lines build a bitcode file for x86 and an object file for whatever the default target is that is running the test. This causes an incompatibility between the files. Add the triple to the llvm-mc invocation.
-
Florian Hahn authored
Add a few test cases with clobbers (writes and lifetime.end) that should prevent memcpy->byval forwarding, but those clobbers are missed at the moment.
-
Andrzej Warzynski authored
This patch updates `validModule` not to use `getModule`. `getModule` will dereference `module`, but that will lead to a seg-fault if `module` is a `nullptr`. Differential Revision: https://reviews.llvm.org/D119705
-
Shengchen Kan authored
Reviewed By: pengfei Differential Revision: https://reviews.llvm.org/D119914
-
Pavel Labath authored
After applying the same for as in TestThreadBacktraceRepeat, the test appears to pass reliably. The skip decorator was added many years ago, so it's not clear whether this is what caused it to hang.
-
Sven van Haastregt authored
It is necessary to guard atomic_double type according to https://www.khronos.org/registry/OpenCL/specs/3.0-unified/html/OpenCL_C.html#_footnotedef_54. Platform that disable cl_khr_int64_base_atomics and cl_khr_int64_extended_atomics will have compiling errors even if atomic_double is not used. Patch by Haonan Yang. Differential Revision: https://reviews.llvm.org/D119398
-
Arlo Siemsen authored
D115300 added Rust as a new PDB language type. This change allows LLDB to recognize the new language type. Reviewed By: rnk Differential Revision: https://reviews.llvm.org/D119044
-
Balazs Benics authored
This is a followup for D118690. I'm removing the offending RUN line, to resolve the build breakage: https://lab.llvm.org/buildbot/#/builders/91/builds/3975/steps/8/logs/stdio Exit Code: 1 Command Output (stderr): -- /scratch/buildbot/bothome/clang-ve-ninja/llvm-project/clang/test/Analysis/analyze-function-guide.cpp:58:22: error: CHECK-EMPTY2-NOT: excluded string found in input // CHECK-EMPTY2-NOT: Every top-level function was skipped. ^ <stdin>:1:1: note: found here Every top-level function was skipped. ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ Differential Revision: https://reviews.llvm.org/D118690
-
Balazs Benics authored
This reverts commit 620d99b7. Let's see if removing the two offending RUN lines makes this patch pass. Not ideal to drop tests but, it's just a debugging feature, probably not that important.
-
Nikita Popov authored
We should not assume that the cet.h header exists just because we're on x86 linux. Only include it if __CET__ is defined. This makes the code more similar to what compiler-rt does in https://github.com/llvm/llvm-project/blob/ee423d93ead39e94c2970b3cc7ef6e6faa75d10b/compiler-rt/lib/builtins/assembly.h#L17 (though that one also has a __has_include() check -- I've not found that to be necessary). Differential Revision: https://reviews.llvm.org/D119697
-
Nikita Popov authored
This is a followup to D119695 using the suggestion by joerg. Rather than manually declaring madvise() on __sun__, this uses posix_madvise() if available, which does get declared properly on Illumos. Differential Revision: https://reviews.llvm.org/D119856
-
Qiu Chaofan authored
These tests are dumped without optimization, which makes them too lengthy and contain meaningless load/stores. Clean them up to prepare for future headers update.
-
Jan Svoboda authored
Commit 4a794d84 caused build failure with -Werror -Wpessimizing-move on the clang-ppc64-aix buildbot. This patch applies Clang's suggestion to remove `std::move`.
-
Zakk Chen authored
The goal is support tail and mask policy in RVV builtins. We focus on IR part first. If the passthru operand is undef, we use tail agnostic, otherwise use tail undisturbed. My plan is to handle more complex operations in follow-up patches. Reviewers: frasercrmck Differential Revision: https://reviews.llvm.org/D118253
-