- Nov 02, 2023
-
-
Ben Shi authored
The passed in parameter 'State' is always identical to 'C.getState()'.
-
Christian Ulmann authored
This commit removes the support for lowering MemRefToLLVM to LLVM dialect with typed pointers. Typed pointers have been deprecated for a while now and it's planned to soon remove them from the LLVM dialect. Related PSA: https://discourse.llvm.org/t/psa-removal-of-typed-pointers-from-the-llvm-dialect/74502
-
Christian Ulmann authored
This commit removes the support for lowering NVGPU to NVVM dialect with typed pointers. Typed pointers have been deprecated for a while now and it's planned to soon remove them from the LLVM dialect. Related PSA: https://discourse.llvm.org/t/psa-removal-of-typed-pointers-from-the-llvm-dialect/74502
-
Weining Lu authored
-
Kazu Hirata authored
Identified with misc-include-cleaner.
-
Yuxuan Chen authored
This PR is proposing a fix for https://github.com/llvm/llvm-project/issues/65971. Previously, given a coroutine like this ``` task foo(int a) { co_return; } ``` Parameter `a` is never used. However, because C++ coroutines move constructs the variable to a heap allocated coroutine activation frame, we considered all parameters referenced. When diagnosing unused parameters, we cannot distinguish if the variable reference was due to coroutine parameter moves. Compiler Explorer shows that GCC warns against this case correctly, but clang does not: https://godbolt.org/z/Wo7dfqeaf This patch addresses this issue by preserving the original `ParmVarDecl`'s `Referenced` state.
-
Chuanqi Xu authored
There are 2 dangling else warnings and a warning pararences around bit operations. This patch tries to eliminate that.
-
Matt Arsenault authored
This reverts commit 6bf1b4e8. Requiring a triple does not require moving these to a codegen test directory. Move these to an x86 specific subdirectory of a transforms test.
-
Fangrui Song authored
[ELF] adjustOutputSections: don't copy SHF_EXECINSTR when an output does not contain input sections (#70911) For an output section with no input section, GNU ld eliminates the output section when there are only symbol assignments (e.g. `.foo : { symbol = 42; }`) but not for `.foo : { . += 42; }` (`SHF_ALLOC|SHF_WRITE`). We choose to retain such an output section with a symbol assignment (unless unreferenced `PROVIDE`). We copy the previous section flag (see https://reviews.llvm.org/D37736) to hopefully make the current PT_LOAD segment extend to the current output section: * decrease the number of PT_LOAD segments * If a new PT_LOAD segment is introduced without a page-size alignment as a separator, there may be a run-time crash. However, this `flags` copying behavior is not suitable for `.foo : { . += 42; }` when `flags` contains `SHF_EXECINSTR`. The executable bit is surprising (https://discourse.llvm.org/t/lld-output-section-flag-assignment-behavior/74359). I think we should drop SHF_EXECINSTR when copying `flags`. The risk is a code section followed by `.foo : { symbol = 42; }` will be broken, which I believe is unrelated as such uses are almost always related to data sections. For data-command-only output sections (e.g. `.foo : { QUAD(42) }`), we keep allowing copyable SHF_WRITE. Some tests are updated to drop the SHF_EXECINSTR flag. GNU ld doesn't set SHF_EXECINSTR as well, though it sets SHF_WRITE for some tests while we don't. -
Noah Goldstein authored
`and` is generally more supported so if we have a `ptrmask` anyways might as well use `and`. Differential Revision: https://reviews.llvm.org/D156640 Closes #67166
-
Noah Goldstein authored
If we have assosiated attributes i.e `([ret_attrs] (ptrmask (ptrmask p0, m0), m1))` we should preserve `[ret_attrs]` when combining the two `llvm.ptrmask`s. Differential Revision: https://reviews.llvm.org/D156638
-
Noah Goldstein authored
We can deduce the former based on the mask / incoming pointer alignment. We can set the latter based if know the result in non-zero (this is essentially just caching our analysis result). Differential Revision: https://reviews.llvm.org/D156636
-
Noah Goldstein authored
Logic basically copies 'and' but we can't return a constant if the result == rhs (mask) so that case is skipped.
-
Noah Goldstein authored
Differential Revision: https://reviews.llvm.org/D156634
-
Noah Goldstein authored
Mostly the same as `and`. We also have a check for a useless `llvm.ptrmask` if the ptr is already known aligned. Differential Revision: https://reviews.llvm.org/D156633
-
Noah Goldstein authored
Differential Revision: https://reviews.llvm.org/D156632
-
Kazu Hirata authored
-
Jakub Kuderski authored
Do not match on illegal destination blocks. Also apply some minor cleanups. TODO: We should add region/block argument type conversions to properly address this. Issue: https://github.com/llvm/llvm-project/issues/70813
-
Kazu Hirata authored
The corresponding function definition was removed by: commit 3787ee45 Author: Nick Desaulniers <ndesaulniers@google.com> Date: Tue Jun 8 08:57:12 2021 -0700
-
Tobias Stadler authored
Missed test case in experimental target, which was not covered by pre-merge checks.
-
saienduri authored
-
Davide Italiano authored
-
Peiming Liu authored
…r operation
-
Valentin Clement (バレンタイン クレメン) authored
-
Tobias Stadler authored
Reland 3686a0b6 after fixing an exposed miscompile in #68840 Differential Revision: https://reviews.llvm.org/D159140
-
Jason Molenda authored
MachProcess has a MachTask as an ivar. In the MachProcess dtor, we call MachTask::Clear() to clear its state, before running the dtor of all our ivars, including the MachTask one. When we attach on darwin, MachProcess calls MachTask::StartExceptionThread which does the task_for_pid and then starts a thread to listen for mach messages. Then MachProcess calls ptrace(PT_ATTACHEXC). If that ptrace() fails, MachProcess will call MachTask::Clear. But the exception thread is now up & running and is not stopped; its ivars will be reset by the Clear() method, and its object will be freed after the dtor runs. Actually eliciting a crash in this scenario is very timing sensitive; I hand-modified debugserver to fail to PT_ATTACHEXC trying to simulate it on my desktop and was unable. But looking at the source, and an occasional crash report we've received, it's clear that this is possible. rdar://117521198
-
philnik777 authored
According to https://developer.apple.com/support/xcode/, quite a few of our availability macros don't do anything anymore, so we might as well remove them to clean up the code a bit.
-
Jon Chesterfield authored
If you build with dynamic_hsa, the symbol is known and compilation succeeds. If you then run with a slightly older libhsa, this argument is not recognised and an error returned. I'd rather the program runs with a misleading omp wtime than refuses to run at all.
-
Davide Italiano authored
zlib. Not everyone wants to checkout from `git`. Tested with and without the env var.
-
Antonio Abbatangelo authored
Revert "Reland "[clang][Sema] Use original template pattern when declaring implicit deduction guides for nested template classes" (#69676)" This reverts commit f4183197. Failing test case: https://github.com/llvm/llvm-project/pull/69676#issuecomment-1789255366
-
nicole mazzuca authored
looks like there may be some missing testing here.
-
Valery Pykhtin authored
Using GCNDownwardRPTracker or GCNUpwardRPTracker the pass collects register pressure values for a function and prints these values next to instructions. Output can be used to generate Filecheck rules in mir tests.
-
Youngsuk Kim authored
Opaque ptr cleanup effort.
-
Louis Dionne authored
This makes it easier to write .sh.cpp tests that perform multiple clang-verify checks in the same test.
-
Jeremy Kun authored
- Document legacy ops lowered by `convert-func-to-llvm` and link it to a GH issue - Document the need for bufferization before `convert-linalg-to-loops` - Document the ops lowered by `expand-strided-metadata`.
-
Craig Topper authored
Needed for -riscv-experimental-rv64-legal-i32 and probably GISel.
-
Youngsuk Kim authored
Opaque ptr cleanup effort. NFC.
-
Graham Inggs authored
Co-authored-by:Sylvestre Ledru <sylvestre@debian.org>
-
Valentin Clement (バレンタイン クレメン) authored
Do not issue a hard error when the function in acc routine directive is not present in the current translation unit. Only issue a warning.
-
Zequan Wu authored
-