- Mar 09, 2024
-
-
Eric Fiselier authored
LIT was never really meant to generate tests during discovery, and we probably shouldn't be doing this. This hack is even worse than the initial attempt because it buries the "UNSUPPORTED" at the bottom of the test.
-
David Spickett authored
535da108 introduced a check, when execve fails, to see if we are allowed to trace programs at all. Unfortunately because we like to call Status vars "error" and "status" in various combinations, one got misnamed. This lead to lldb-server trying to make an error value out of a success value when you did the following: ``` $ ./bin/lldb-server gdbserver 127.0.0.1:1234 -- is_not_a_file Assertion failed: (Err && "Cannot create Expected<T> from Error success value."), function Expected... ``` This happened because the execve fails, but the check whether we can trace says yes we can trace, but then we use the Status from the check to create the return value. That Status is in fact a success value not the failed Status we got from the execve attempt. With the name corrected you now get: ``` $ ./bin/lldb-server gdbserver 127.0.0.1:1234 -- is_not_a_file error: failed to launch 'is_not_a_file': execve failed: No such file or directory ``` Which is what we expect to see. This also fixes the test `TestGDBRemoteLaunch.py` when asserts are enabled.
-
lntue authored
-
Louis Dionne authored
It looks like that broke the CI even more. Reverting until I can investigate.
-
Nick Desaulniers authored
-
erichkeane authored
While working on a followup patch, I discovered we didn't serialize/deserialize the Compute Context properly, so this patch fixes it, then adds a PCH mode to the ast test.
-
Boian Petkantchin authored
Make them more general instead of only supporting `func::FuncOp`.
-
Jonathan Thackray authored
Add support for Arm Cortex A78AE CPU Technical Reference Manual for Arm Cortex A78AE: https://developer.arm.com/documentation/101779/0003 Fixes #84450
-
Andreas Fertig authored
Fix #84064 According to http://eel.is/c++draft/dcl.fct.def.coroutine#9 the first parameter for overload resolution of `operator new` is `size_t` followed by the arguments of the coroutine function. http://eel.is/c++draft/dcl.fct.def.coroutine#4 states that the first argument is the lvalue of `*this` if the coroutine is a member function. Before this patch, Clang handled class types correctly but ignored lambdas. This patch adds support for lambda coroutines with a `promise_type` that implements a custom `operator new`. The patch does consider C++23 `static operator()`, which already worked as there is no `this` parameter.
-
Jay Foad authored
Use optional arguments instead of separate (multi)classes for renamed instructions.
-
Louis Dionne authored
Those have been crippling the CI for over a week now. This is the only solution I see until we gain a better understanding of why they fail. Otherwise all other patches are blocked on these spurious failures in the stage1 of the CI.
-
Antonio Frighetto authored
-
- Mar 08, 2024
-
-
Yaxun (Sam) Liu authored
In -fgpu-rdc mode, when an external kernel is used by a host function with weak_odr linkage (e.g. explicitly instantiated template function), the kernel should not be marked as host-used external kernel, since the host function may be dropped by the linker. Mark the external kernel as host-used external kernel will force a reference to the external kernel, which the user may not define in other TU. Fixes: https://github.com/llvm/llvm-project/issues/83771
-
Schrodinger ZHU Yifan authored
-
Lukacma authored
This patch adds clang and llvm support for following intrinsic and maps it to DUPQ instruction: ``` // Variants are also available for: // _s8, _u16, _s16, _u32, _s32, _u64, _s64 // _bf16, _f16, _f32, _f64 svuint8_t svdup_laneq[_u8](svuint8_t zn, uint64_t imm_idx); ```
-
Andreas Jonson authored
implementation as discussed in https://discourse.llvm.org/t/rfc-metadata-attachments-for-function-arguments/76420
-
Krzysztof Parzyszek authored
When walking over functions (in pre-order), if the function being visited needs to be erased, skip visiting its regions. This was detected by address sanitizer.
-
Simon Pilgrim authored
When we copied the SkylakeServer model we missed this diff Confirmed with uops.info and Agner
-
Joseph Huber authored
Summary: A lot of these tests implicitly rely on `--no-offload-new-driver` a patch in the future will change the default so this patch makes it explicit in locations where it matters.
-
Guillaume Chatelet authored
Umbrella bug #83182
-
Stephen Tozer authored
This patch adds minimal documentation for the IR representation of the RemoveDIs model of debug info. This patch assumes that the default for all cases is still debug intrinsic functions, and so does not update all existing text to refer to debug records, but only adds a section in the LangRef and SourceLevelDebugging documents to explain the new format.
-
Will Hawkins authored
Almost every test needed a CanBePiped concept and each implemented it separately, but identically. Consolidate all implementations into test_range.h.
-
Balazs Benics authored
This fixups 7457e2c1 as I forgot one switch to update, which then broke this build bot: https://lab.llvm.org/buildbot/#/builders/57/builds/33272
-
AtariDreams authored
-
AtariDreams authored
This only converts the instances where all that is needed is to change the variable type name. Basically, anything that involves a function that LiveRegUnits does not directly have was skipped to play it safe. Reverts https://github.com/llvm/llvm-project/commit/7a0e222a17058a311b69153d0b6f1b4459414778
-
paperchalice authored
Like ModuleToFunctionPassAdaptor. --------- Co-authored-by:Matt Arsenault <arsenm2@gmail.com>
-
Haojian Wu authored
Fixes #54051 This patch implements the C++20 feature -- CTAD for alias templates (P1814R0, specified in https://eel.is/c++draft/over.match.class.deduct#3). It is an initial patch: - it cover major pieces, thus it works for most cases; - the big missing piece is to implement the associated constraints (over.match.class.deduct#3.3) for the synthesized deduction guides, see the FIXME in code and tests; - Some enhancements on the TreeTransform&TemplateInstantiator to allow performing instantiation on `BuildingDeductionGuides` mode;
-
Krzysztof Parzyszek authored
….h.inc The consumers of OpenMP-related definitions include a TableGen-generated file OMP.h.inc. Having a separate OMP.h allows putting additional declarations in there that are not auto-generated. This patch is NFC.
-
Krzysztof Parzyszek authored
When` SmallVector<char>` is used as a backing storage, it can't be assumed to end with a \x0. When creating a `StringRef` from it, pass the length explicitly. This was detected by address sanitizer.
-
Krzysztof Parzyszek authored
-
Krzysztof Parzyszek authored
Rewriting an op can invalidate the operator range being iterated on. Store the users in a separate list, and iterate over the list instead. This was detected by address sanitizer.
-
Congcong Cai authored
[clang-tidy]avoid bugprone-unused-return-value false positive for function with the same prefix as the default argument (#84333)
-
Guillaume Chatelet authored
Its sole user is `BigInt` so moving `full_mul` inside UInt.h.
-
Guillaume Chatelet authored
-
Balazs Benics authored
Note that this patch will be necessary to fix `forEachArgumentWithParam()` and `forEachArgumentWithParamType()` matchers for deducing "this"; which is my true motivation. There the bug is that with explicit obj params, one should not adjust the number of arguments in presence of `CXXMethodDecls`, and this causes a mismatch there mapping the argument to the wrong param. But, I'll come back there once we have this matcher.
-
Joseph Huber authored
Summary: This default enum is used in other places, we should keep it consistent.
-
Shengchen Kan authored
APX assembly syntax recommendations: https://cdrdv2.intel.com/v1/dl/getContent/817241 NOTE: The change in llvm/tools/llvm-exegesis/lib/X86/Target.cpp is for test LLVM :: tools/llvm-exegesis/X86/latency/latency-SETCCr-cond-codes-sweep.s For `SETcc`, llvm-exegesis would randomly choose 1 other instruction to test with `SETcc`, after selecting the instruction, llvm-exegesis would check if the operand is initialized and valid, if not `randomizeTargetMCOperand` would choose a value for invalid operand, it misses support for condition code operand, which cause the flaky failure after `CCMP` supported. llvm-exegesis can choose `CCMP` without specifying ccmp feature b/c it use `MCSubtarget` and only16/32/64 bit is considered. llvm-exegesis doesn't choose other instructions b/c requirement in `hasAliasingRegistersThrough`: the instruction should use GPR (defined by `SETcc`) and define `EFLAGS` (used by `SETcc`).
-
Kiran Chandramohan authored
Fixes #84088
-
Jay Foad authored
-
Younan Zhang authored
We don't modify the MLTAL parameter in `SetupConstraintScope`, and it is better if we don't copy the 120-byte object each time we call the function.
-