- Dec 22, 2021
-
-
Igor Kudrin authored
This is similar to what we have already done to some other tests. See D102643, D102710, D102754. Differential Revision: https://reviews.llvm.org/D116108
-
Igor Kudrin authored
dwarfgen::Generator cannot be created if there is no asm backend for a target. For example, if the default target triple is nvptx-nvidia-cuda, some tests fail even after D98400, which added checks for most cases. This patch extends the approach to the remaining cases. Differential Revision: https://reviews.llvm.org/D116107
-
Igor Kudrin authored
If Generator::create() returns an error, tests should fail gracefully and report the cause, for example: [ RUN ] DebugLineBasicFixture.ParserSkipsCorrectly .../llvm/unittests/DebugInfo/DWARF/DWARFDebugLineTest.cpp:47: Failure Value of: llvm::detail::TakeExpected(ExpectedGenerator) Expected: succeeded Actual: failed (no asm backend for target nvptx64-nvidia-cuda) Differential Revision: https://reviews.llvm.org/D116106
-
Adrian Kuegel authored
-
Nikita Popov authored
Mark the first function optnone as well, to make sure that the test is independent of optimization.
-
Adrian Kuegel authored
-
Nikita Popov authored
As reames mentioned on related reviews, we don't need the nocapture requirement here. First of all, from an API perspective, this is not something that MemoryLocation::getForDest() should be checking in the first place, because it does not affect which memory this particular call can access; it's an orthogonal concern that should be handled by the caller if necessary. However, for both of the motivating users in DSE and InstCombine, we don't need the nocapture requirement, because the capture can either be purely local to the call (a pointer identity check that is irrelevant to us), be part of the return value (which we check is unused), or be written in the dest location, which we have determined to be dead. This allows us to remove the special handling for libcalls as well. Differential Revision: https://reviews.llvm.org/D116148
-
Serge Pavlov authored
This are tests extracted from https://reviews.llvm.org/D110322, committed prior to that patch to show the change in behavior.
-
Jun Zhan authored
This patch implements __builtin_reduce_xor as specified in D111529. Reviewed By: fhahn, aaron.ballman Differential Revision: https://reviews.llvm.org/D115231
-
Diana Picus authored
descriptor.h is using std::max, so it should include <algorithm>. This should fix a build issue on Windows on Arm.
-
Krasimir Georgiev authored
The old member was `= 0`: https://github.com/llvm/llvm-project/commit/f5ac23b5ae090d64d31f0b6624470af97dc20bf6#diff-7781d63141d53242da0520361a554df579ecb079b33bdcfbe7a0db95d44cca49L1740 It looks like this caused some bots to fail: https://lab.llvm.org/buildbot/#/builders/127/builds/21684
-
Nikita Popov authored
It is not necessary to explicitly check which attributes are present, and only add those to the builder. We can simply list all attributes that need to be stripped and remove them unconditionally. This also allows us to use some nicer APIs that don't require mucking about with attribute list indices.
-
Nikita Popov authored
We're only removing a single attribute, so there is no need to go through AttrBuilder.
-
Nikita Popov authored
As we're only removing and adding a single attribute, there is no need to go through AttrBuilder.
-
Nikita Popov authored
The areFunctionArgsABICompatible() hook currently accepts a list of pointer arguments, though what we're actually interested in is the ABI compatibility after these pointer arguments have been converted into value arguments. This means that a) the current API is incompatible with opaque pointers (because it requires inspection of pointee types) and b) it can only be used in the specific context of ArgPromotion. I would like to reuse the API when inspecting calls during inlining. This patch converts it into an areTypesABICompatible() hook, which accepts a list of types. This makes the method more generally usable, and compatible with opaque pointers from an API perspective (the actual usage in ArgPromotion/Attributor is still incompatible, I'll follow up on that in separate patches). Differential Revision: https://reviews.llvm.org/D116031
-
Max Kazantsev authored
-
Jason Molenda authored
Version 2 of 'main bin spec' LC_NOTE allows for the specification of a slide of where the binary is loaded in the corefile virtual address space. It also adds a (currently unused) platform field for the main binary. Some corefile creators will only have a UUID and an offset to be applied to the binary. Changed TestFirmwareCorefiles.py to test this new form of 'main bin spec' with a slide, and also to run on both x86_64 and arm64 macOS systems. Differential Revision: https://reviews.llvm.org/D116094 rdar://85938455
-
Serge Pavlov authored
According to the discussion in https://reviews.llvm.org/D110322 the code that removes side effect from replaced function call is deleted. Differential Revision: https://reviews.llvm.org/D115870
-
Kazu Hirata authored
-
Fangrui Song authored
This does not decrease sizeof(InputSection) (important for memory usage) on ELF64 by itself but allows we to add another uint32_t.
-
Chuanqi Xu authored
-
Chuanqi Xu authored
This commit adds two test about template class instantiation in transitively imported module. They are used as pre-commit tests for successive patches. Differential Revision: https://reviews.llvm.org/D116097
-
LLVM GN Syncbot authored
-
Nikolas Klauser authored
Granularize the `<filesystem>` header Reviewed By: Quuxplusone, ldionne, #libc Spies: libcxx-commits, mgorny Differential Revision: https://reviews.llvm.org/D115578
-
Arthur O'Dwyer authored
Eliminate a bogus operator== overload. Also, check more intermediate steps in the logic we're checking here. Some of this simplification is possible only now that we've implemented more of <ranges>. Differential Revision: https://reviews.llvm.org/D116002
-
Julian Lettner authored
See also: https://reviews.llvm.org/D111157
-
Owen Pan authored
Move the handling of brace wrapping after => from unwrapped line parser to token annotator and clean up the parser. Differential Revision: https://reviews.llvm.org/D115967
-
John Ericson authored
Extracted from D99484. My new plan is to start from the outside and work inward. Reviewed By: compnerd Differential Revision: https://reviews.llvm.org/D115570
-
Alexandre Ganea authored
Reland integrates build fixes & further review suggestions. Thanks to @zturner for the initial S_OBJNAME patch! Differential Revision: https://reviews.llvm.org/D43002
-
Alexandre Ganea authored
Also revert all subsequent fixes: - abd1cbf5 [Clang] Disable debug-info-objname.cpp test on Unix until I sort out the issue. - 00ec4412 [Clang] debug-info-objname.cpp test: explictly encode a x86 target when using %clang_cl to avoid falling back to a native CPU triple. - cd407f6e [Clang] Fix build by restricting debug-info-objname.cpp test to x86.
-
Nico Weber authored
-
Martin Storsjö authored
If %{exec} sets "--env PATH=single-dir", the directory containing bash and related shell utils is omitted from the path, which means that most shell scripts would fail. (Setting PATH is needed for DLL builds on Windows; PATH fills the same role as e.g. LD_LIBRARY_PATH on Linux.) This condition is missed in the current test, because the executor run.py first resolves the executable to run using the original path, then invokes that executable with an environment with a restricted path. Thus the executor is able to run bash, but that bash is then unable to run further shell commands (other than bash builtins). Extend the test from "bash --version" to "bash -c 'bash --version'". This correctly identifies the executor-has-no-bash condition in the current Windows CI configs, allowing removing 6 cases of LIBCXX-WINDOWS-FIXME. Another longterm fix would be to extend run.py with an option like "--env-prepend PATH=dir", to allow keeping the current p... -
Martin Storsjö authored
This is similar to the existing setting LIBCXX_ABI_DEFINES, with the difference that this also allows setting other defines than ones that start with "_LIBCPP_ABI_", and allows setting defines to a specific value. This allows avoiding using LIBCXX_TEST_COMPILER_FLAGS in two CI configurations. Differential Revision: https://reviews.llvm.org/D116109
-
Martin Storsjö authored
This allows cross-testing (by setting LIBCXX_EXECUTOR to point to ssh.py) without making an entirely new test config file. Implicitly, this also fixes quoting of the python executable name (which is quoted in test/CMakeLists.txt). Differential Revision: https://reviews.llvm.org/D115398
-
Alexandre Ganea authored
Fixes PR52704 : https://github.com/llvm/llvm-project/issues/52704 Differential Revision: https://reviews.llvm.org/D116011
-
Louis Dionne authored
This moves the macro definitions back to __config, but keeps the improved documentation. 346ef5e5 had broken the MinGW build.
-
Kirill Stoimenov authored
Making callbacks hidden will remove PLT calls. Reviewed By: vitalybuka Differential Revision: https://reviews.llvm.org/D116121
-
Nemanja Ivanovic authored
In commit 1674d9b6, I missed adding the updates to existing test cases. This should bring the bots back to green.
-
Louis Dionne authored
Also, move the setting of the macro closer to its point of use, which also has the benefit of uncluttering `__config`.
-
Nemanja Ivanovic authored
The current code makes the assumption that equality comparison can be performed with a word comparison instruction. While this is true if the entire 64-bit results are used, it does not generally work. It is possible that the low order words and high order words produce different results and a user of only one will get the wrong result. This patch adds an and of the result words so that each word has the result of the comparison of the entire doubleword that contains it. Differential revision: https://reviews.llvm.org/D115678
-