- Sep 24, 2021
-
-
Simon Pilgrim authored
Testing on a SLM box suggests these can run on either port, but the throughput is 4cy on either (inc MMX versions). Confirmed with Intel AoM / Agner / InstLatX64.
-
Simon Pilgrim authored
Record is a SmallVector<uint64_t, 1024> - we really need to avoid passing this by value. Avoid unnecessary big copies, reported by coverity.
-
David Spickett authored
Earlier versions of GDB do not do this automatically. (from my checks 8.3 does not and 9.2 does)
-
David Sherwood authored
There are several places in the code that are currently broken as they assume an Instruction always has a parent Function when attempting to get the vscale_range attribute. This patch adds checks that an Instruction has a parent. I've added a test for a parentless @llvm.vscale intrinsic call here: unittests/Analysis/ValueTrackingTest.cpp Differential Revision: https://reviews.llvm.org/D110158
-
gbreynoo authored
This change is to keep the help text and command guide of objcopy in tandem. - In the help output the options --rename-section and --set-section-flags were missing the flag exclude, which is found in the command guide. - In the command guide the alias -G for --keep-global-symbol was missing, which is found in the help output. Differential Revision: https://reviews.llvm.org/D110340 -
Dmitri Gribenko authored
-
David Spickett authored
I found this after upgrading from Ubuntu bionic (gdb 8.1.1) to Focal (gdb 9.2). (where this test fails, but that's for a different patch) 9.2 allows you to set breakpoint commands from Python, which was added in 8.3. (bintutils a913fffbdee21fdd50e8de0596358be425775678 "Allow breakpoint commands to be set from Python") The reason this test never failed before was because it did so silently. "source <python file>" doesn't fail even if that script raises an Exception. To fix this extend the gdb lit feature to check that: * gdb exists * has Python support * allows you to set breakpoint commands Reviewed By: #libc, ldionne Differential Revision: https://reviews.llvm.org/D110334
-
Jonas Paulsson authored
The type legalizer has by default no method of doing this bitcast other than storing and reloading the value from stack. This patch implements a custom lowering of this operation using extractions of subregs (z13 and earlier using FP128 register pairs), or of vector elements (with 'vector enhancements 1' using VR128 FP registers). Review: Ulrich Weigand Differential Revision: https://reviews.llvm.org/D110346
-
David Green authored
Instead of relying on many volatile loads/stores in a single function, rewrite the test to use separate functions as any other test would.
-
Matthias Springer authored
* Do not discard static result type information that cannot be inferred from lower/upper padding. * Add optional argument to `PadTensorOp::inferResultType` for specifying known result dimensions. Differential Revision: https://reviews.llvm.org/D110380
-
Petr Hosek authored
When statically linking C++ standard library, we shouldn't add -Bdynamic after including the library on the link line because that might override user settings like -static and -static-pie. Rather, we should surround the library with --push-state/--pop-state to make sure that -Bstatic only applies to C++ standard library and nothing else. This has been supported since GNU ld 2.25 (2014) so backwards compatibility should no longer be a concern. Differential Revision: https://reviews.llvm.org/D110128
-
Amara Emerson authored
Odd switch case types cause a crash in the conversion to MVT. Instead use a pointer sized scalar type which is what SDAG does in these cases.
-
Ludovic Jozeau authored
Fix little inconsistency and use `std::string` (which is used everywhere else) instead of `string` Reviewed By: MyDeveloperDay, HazardyKnusperkeks Differential Revision: https://reviews.llvm.org/D108765
-
mydeveloperday authored
[clang-format] ensure clang-format command-line argument sets up the default left/right qualifier ordering When specifying the alignment direction on the command line ensure we set up the default ordering. Fix spelling mistakes in the command-line argument Reviewed By: HazardyKnusperkeks Differential Revision: https://reviews.llvm.org/D110359
-
Pavel Labath authored
We need a different flag combination to produce .debug_frame.
-
mydeveloperday authored
-
Amara Emerson authored
It was assuming that GPR extends could only have destination sizes of 32 or 64 bits, but for AArch64 we allow < 32 bits even without matching size physregs.
-
Lang Hames authored
This is an ORC-runtime counterpart to LLVM commit ef391df2, and the motivation is the same: to move to a shorter name to improve the ergonomics of this type before it's more widely adopted.
-
Albion Fung authored
The __darn family of builtins are only available on Pwr9, and only __darn_32 is available on both 64 and 32 bit, while the rest are only available on 64 bit. The patch adds sema checking for these builtins and separate the __darn_32's 32 bit test cases. Differential revision: https://reviews.llvm.org/D110282
-
David Blaikie authored
-
David Blaikie authored
-
David Blaikie authored
-
Lang Hames authored
Removing the 'ess' suffix improves the ergonomics without sacrificing clarity. Since this class is likely to be used more frequently in the future it's worth some short term pain to fix this now.
-
Frederic Cambus authored
Reviewed By: aeubanks, asbirlea Differential Revision: https://reviews.llvm.org/D109663
-
David Carlier authored
and ttyentpath separately on NetBSD. Reviewed By: vitalybuka Differential Revision: https://reviews.llvm.org/D109843
-
Christudasan Devadasan authored
Tablegen currently expects targets to have at least one pressure set for every broader register category. AMDGPU's VGPR or AGPR, for instance, seemed to work correctly without any pset, though we have forced one for each type to avoid the assertion in computeRegUnitSets. However, psets can not be entirely empty. At least one set is mandatory for every target. This patch bypasses the assertion for the classes when GeneratePressureSet is zero while ensuring the RegUnitSets are not empty. Reviewed By: arsenm, rampitec Differential Revision: https://reviews.llvm.org/D110305
-
LLVM GN Syncbot authored
-
Lang Hames authored
EPCGenericDylibManager provides an interface for loading dylibs and looking up symbols in the executor, implemented using EPC-calls to functions in the executor. SimpleExecutorDylibManager is an executor-side service that provides the functions used by EPCGenericDylibManager. SimpleRemoteEPC is updated to use an EPCGenericDylibManager instance to implement the ExecutorProcessControl loadDylib and lookup methods. In a future commit these methods will be removed, and clients updated to use EPCGenericDylibManagers directly.
-
David Blaikie authored
This excludes certain names that can't be rebuilt from the available DWARF: * Atomic types - no DWARF differentiating int from atomic int. * Vector types - enough DWARF (an attribute on the array type) to do this, but I haven't written the extra code to add the attributes required for this * Lambdas - ambiguous with any other unnamed class * Unnamed classes/enums - would need column info for the type in addition to file/line number * noexcept function types - not encoded in DWARF
-
Christudasan Devadasan authored
We don't allow an initializer for LDS variables and there is an early abort during instruction selection. This patch legalizes them by ignoring the init values. During assembly emission, proper error reporting already exists for such instances. Reviewed By: arsenm Differential Revision: https://reviews.llvm.org/D109901
-
Fangrui Song authored
to match GCC and Clang's own x86-64.
-
Michael Kruse authored
This is a simple version without the possibility to define distribute points or followup-transformations. However, it is the first transformation that has to check whether the transformation is correct. It interprets the same metadata as the LoopDistribute pass. Re-apply after revert in c7bcd72a with fix: Take isBand out of #ifndef NDEBUG since it now is used unconditionally.
-
Mehdi Amini authored
This is only noticeable when using an attribute across dialects I think. Previously the namespace would be ommited, but it wouldn't matter as long as the generated code stays within a single namespace. Differential Revision: https://reviews.llvm.org/D110367
-
Matthias Springer authored
Only scf.for was supported until now. Differential Revision: https://reviews.llvm.org/D110220
-
Matthias Springer authored
Passes such as `linalg-tile-to-tiled-loop` are merged into `linalg-tile`. Differential Revision: https://reviews.llvm.org/D110214
-
Teresa Johnson authored
Fix bot failures after 2c1defee. The new GUID I added isn't matching because it is a local with the source path prepended. There isn't much use in matching the GUID's exactly anyway, so remove those from the patterns.
-
Teresa Johnson authored
In ThinLTO for locals we normally compute the GUID from the name after prepending the source path to get a unique global id. SamplePGO indirect call profiles contain the target GUID without this uniquification, however (unless compiling with -funique-internal-linkage-names). Therefore, the index contains the original GUID of the local symbols (without module path prepended to uniquify), in order to correctly handle the call edges added for these indirect call profile targets with SamplePGO. We were emitting these to the combined index when writing it out as bitcode, which is unnecessary and causes overhead when writing out the indexes for distributed backends. The only use of the original GUID name is in the thin link. Suppress it in that case. This reduced the thin link time for a large distributed build by about 7%, and the aggregate size of the serialized indexes by over 2%. Continue to print it when writing out the full index, since that is just used for debugging and testing. Update a distributed thinlto index test to contain a local and ensure that we don't get a COMBINED_ORIGINAL_NAME record. Differential Revision: https://reviews.llvm.org/D110296
-
Leonard Chan authored
This test specifically checks that profiles are not mergeable if there's a change in the CounterPtr in the profile header. The test manually changes CounterPtr by explicitly calling memset on some offset into the profile file. This test would fail if binary IDs were emitted because the offset calculation does not take into account the binary ID sizes. This patch updates the test to use types provided in profile/InstrProfData.inc to make it more resistant to profile layout changes. Differential Revision: https://reviews.llvm.org/D110277
-
Lang Hames authored
This should have been included with ExecutorBootstrapService in 78b083db, but was accidentally left out. It give services a chance to release any resources that they have acquired.
-
Nemanja Ivanovic authored
Commit a44ab170 added a unit test that fails to build with -Werror which causes build bot breaks on bots that include that option in their build. This patch just adds the necessary casts to silence the warnings.
-