- Jun 05, 2020
-
-
Nikita Popov authored
Stores now always have an alignment.
-
Nikita Popov authored
Loads and stores always have an alignment now.
-
Marco Elver authored
Summary: Use a portable section name, as for the test's purpose any name will do. Reviewers: nickdesaulniers, thakis Reviewed By: thakis Subscribers: cfe-commits Tags: #clang Differential Revision: https://reviews.llvm.org/D81306
-
Nico Weber authored
This reverts commit 74bd9882. Breaks LLVM::section-headers.test everywhere, see e.g. http://lab.llvm.org:8011/builders/clang-x86_64-debian-fast/builds/29940/steps/test-check-all/logs/FAIL%3A%20LLVM%3A%3Asection-headers.test
-
Adrian Prantl authored
Since FindXcodeContentsDirectoryInPath expects the *.app/Contents and DEVELOPER_DIR is supposed to point to Xcode.app, we need to append the Contents path first. Differential Revision: https://reviews.llvm.org/D81290
-
Adrian Prantl authored
Differential Revision: https://reviews.llvm.org/D81289
-
Craig Topper authored
Similar to D81212. Differential Revision: https://reviews.llvm.org/D81292
-
Reid Kleckner authored
In my use case, this saved 100ms of time doing one-time-initialization for std::error_code().
-
Stanislav Mekhanoshin authored
Differential Revision: https://reviews.llvm.org/D80749
-
Reid Kleckner authored
Clang marks calls to operator new as heap allocation sites, but the operator declared at global scope returns a void pointer. There is no explicit cast in the code, so the compiler has to write down the allocated type itself. Also generalize a cast to use CallBase, so that we mark heap alloc sites when exceptions are enabled. Reviewed By: MaskRay Differential Revision: https://reviews.llvm.org/D80966
-
LLVM GN Syncbot authored
-
Stanislav Mekhanoshin authored
If an instruction is erased we also need to remove it from Visited set. There is a very small chance that an another newly created instruction will be created with the same pointer value in place of an erased one. Differential Revision: https://reviews.llvm.org/D80958
-
Yaxun (Sam) Liu authored
This reverts commit 11d06b95.
-
Yaxun (Sam) Liu authored
-
Saleem Abdulrasool authored
This merges the static and shared library and behaves as if `-search_paths_first` was specified which is also the default behaviour on ld64 (and now lld). Unify the paths, and use `llvm::sys::path` to deal with the path to be truly agnostic to the host.
-
Alexandre Ganea authored
Before this patch, llvm-pdbutil supported only --type-stats to dump stats about a PDB TPI stream. Adding --id-stats for completion.
-
Matt Arsenault authored
-
Adrian Prantl authored
instead of preferring the one chosen with xcode-select. <rdar://problem/64000666> Differential Revision: https://reviews.llvm.org/D81210
-
Adrian Prantl authored
These functions really don't belong into PlatformDarwin, since they actualy query state of the Host and not of the remote platform.
-
Matt Arsenault authored
Just computing the alignment makes sense without caring about the general known bits, such as for non-integral pointers. Separate the two and start calling into the TargetLowering hooks for frame indexes. Start calling the TargetLowering implementation for FrameIndexes, which improves the AMDGPU matching for stack addressing modes. Also introduce a new hook for returning known alignment of target instructions. For AMDGPU, it would be useful to report the known alignment implied by certain intrinsic calls. Also stop using MaybeAlign.
-
Matt Arsenault authored
Immediate legalize fields were added for G_SEXT_INREG. Simiarly, these are likely not legal except for certain offsets.
-
Dan Gohman authored
It broke clang-check. This reverts commit 931fcd3b.
-
Jay Foad authored
Related to https://reviews.llvm.org/D80916 Differential Revision: https://reviews.llvm.org/D81271
-
Fangrui Song authored
-
Nico Weber authored
-
Yaxun (Sam) Liu authored
-
Matt Arsenault authored
There were additional standalone patterns for these nodes which were missing the subtarget predicate.
-
Matt Arsenault authored
-
Craig Topper authored
Shifts are supposed to always shift in zeros or sign bits regardless of their inputs. It's possible the input value may have been replaced with undef by SimplifyDemandedBits, but the shift in zeros are still demanded. This issue was reported to me by ispc from 10.0. Unfortunately their failing test does not fail on trunk. Seems to be because the shl is optimized out earlier now and doesn't become VSHLI. ispc bug https://github.com/ispc/ispc/issues/1771 Differential Revision: https://reviews.llvm.org/D81212
-
Dan Gohman authored
This patch addresses the review comments on r352930: - Removes redundant diagnostic checking code - Removes errnoneous use of diag::err_alias_is_definition, which turned out to be ineffective anyway since functions can be defined later in the translation unit and avoid detection. - Adds a test for various invalid cases for import_name and import_module. Differential Revision: https://reviews.llvm.org/D59520
-
Evgenii Stepanov authored
Summary: Non-zero malloc fill is causing way too many hard to debug issues. Reviewers: kcc, pcc, hctim Subscribers: #sanitizers, llvm-commits Tags: #sanitizers Differential Revision: https://reviews.llvm.org/D81284
-
Marco Elver authored
Summary: This makes -fsanitize=kernel-address emit the correct globals constructors for the kernel. We had to do the following: - Disable generation of constructors that rely on linker features such as dead-global elimination. - Only emit constructors for globals *not* in explicit sections. The kernel uses sections for special globals, which we should not touch. Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=203493 Tested: 1. With 'clang/test/CodeGen/asan-globals.cpp'. 2. With test_kasan.ko, we can see: BUG: KASAN: global-out-of-bounds in kasan_global_oob+0xb3/0xba [test_kasan] Reviewers: glider, andreyknvl Reviewed By: glider Subscribers: cfe-commits, nickdesaulniers, hiraditya, llvm-commits Tags: #llvm, #clang Differential Revision: https://reviews.llvm.org/D80805
-
Nikita Popov authored
This should dump to the provided Out, rather than dbgs(), though they coincide in current usage.
-
Nikita Popov authored
PendingInLocs ends up having the same value as InLocs, just computed a bit more indirectly. It is a leftover of a previous implementation approach. This patch drops PendingInLocs, as well as the Diff and Removed calulations, which are no longer needed. Differential Revision: https://reviews.llvm.org/D80868
-
Matt Arsenault authored
Different tools seem to be more or less strict with the markers between different functions.
-
Matt Arsenault authored
Match SelectionDAG's behavior of adding nofpexcept to out instructions that may raise fp exceptions that are selected from instructions that do not.
-
Alex Lorenz authored
The in-process CC1 currently doesn't interoperate with the macOS crash analytics, which we would like to keep enabled for Apple clang. This commit restores the out-of-process CC1 to the Apple clang CMake configuration for now. Differential Revision: https://reviews.llvm.org/D80849
-
Sameer Arora authored
Reorder `DumpSection` under `handleArgs` in file `MachOObjcopy.cpp`. The operation to dump a section is now performed before both add and remove section operations for MachO file format. Change for the ELF format at D81097. Together fixes https://bugs.llvm.org/show_bug.cgi?id=44283 Reviewed By: jhenderson Differential Revision: https://reviews.llvm.org/D81123
-
Nicolas Vasilache authored
Two tests regularly show up on the long tail when testing MLIR. This revision reduces their size.
-
Sameer Arora authored
Reorder `DumpSection` under `handleArgs` in file `ELFObjcopy.cpp`. `DumpSection` is placed before `replaceAndRemoveSections` and is therefore now the first operation under `handleArgs`. Thus, it is now performed before both `add` and `remove` section operations. Change for the MachO format at D81123. Together fixes https://bugs.llvm.org/show_bug.cgi?id=44283. Reviewed By: alexshap, jhenderson, MaskRay Differential Revision: https://reviews.llvm.org/D81097
-