- Oct 23, 2023
-
-
Kazu Hirata authored
-
Wang Pengcheng authored
MultiClass argument is not used any more since aa843266. Besides, for maintainability, we should put the implementation of qualifying name in one place (that is `QualifyName` function), so `Scoper` is removed and we use `IsMC` to indicate that we are in a multiclass.
-
Wang Pengcheng authored
To match GCC's behaviors. Fixes #67596
-
Chuanqi Xu authored
[C++20] [Modules] [Driver] Don't enable -fdelayed-template-parsing by default on windows with C++20 (#69431) There are already 3 issues about the broken state of -fdelayed-template-parsing and C++20 modules: - https://github.com/llvm/llvm-project/issues/61068 - https://github.com/llvm/llvm-project/issues/64810 - https://github.com/llvm/llvm-project/issues/65027 The problem is more complex than I thought. I am not sure how to fix it properly now. Given the complexities and -fdelayed-template-parsing is actually an extension to support old MS codes, I think it may make sense to not enable the -fdelayed-template-parsing option by default with C++20 modules to give more user friendly experience. Users who still want -fdelayed-template-parsing can specify it explicitly. Also according to https://learn.microsoft.com/en-us/cpp/build/reference/permissive-standards-conformance?view=msvc-170, MSVC actually defaults to -fno-delayed-template-parsing (/Zc:twoPhase- with MSVC CLI) if using C++20. So we match the behavior with MSVC here to not enable -fdelayed-template-parsing by default after C++20.
-
Johannes Doerfert authored
-
Matthias Springer authored
Add a new attribute `bufferization.manual_deallocation` that can be attached to allocation and deallocation ops. Buffers that are allocated with this attribute are assigned an ownership of "false". Such buffers can be deallocated manually (e.g., with `memref.dealloc`) if the deallocation op also has the attribute set. Previously, the ownership-based buffer deallocation pass used to reject IR with existing deallocation ops. This is no longer the case if such ops have this new attribute. This change is useful for the sparse compiler, which currently deallocates the sparse tensor buffers by itself.
-
Hongtao Yu authored
Tweaking warnings more to avoid flooding user log.
-
Fangrui Song authored
When linking an executable with a slightly larger executable, ld.lld --call-graph-profile-sort=cdsort can be very slow (see #68638). ``` 4.6% 20.7Mi .text.hot 3.5% 15.9Mi .text 3.4% 15.2Mi .text.unknown ``` Add cl option `cdsort-max-chain-size`, which is similar to `ext-tsp-max-chain-size`, and set it to 128, to improve performance. In `ld.lld @response.txt --threads=4 --call-graph-profile-sort=cdsort --time-trace" builds, the "Total Sort sections" time is measured as follows: * -mllvm -cdsort-max-chain-size=64: 1.321813 * -mllvm -cdsort-max-chain-size=128: 2.030425 * -mllvm -cdsort-max-chain-size=256: 2.927684 * -mllvm -cdsort-max-chain-size=512: 5.493106 * unlimited: 9 minutes The rest part takes 6.8s.
-
Fangrui Song authored
Using the legacy PM for the optimization pipeline was deprecated in 13.0.0. Many legacy passes have been removed in 2022.
-
cmtice authored
lldb/test/Shell/Breakpoint/breakpoint-command.test adds a python command, to be executed when a breakpoint hits, that writes out a number. It then runs, hits the breakpoint and checks that the number is present exactly once. The problem is that on some systems the test can be run in a filepath that happens to contain the number (e.g. auto-generated directory names). The number is then detected multiple times and the test fails. This patch fixes the issue by using a string instead, particularly a string with spaces, which is very unlikely to be auto-generated by any system.
-
Kazu Hirata authored
Identified with misc-include-cleaner.
-
Justin Fargnoli authored
[mlir][DeadCodeAnalysis] Don't Require `RegionBranchTerminatorOpInterface` in `visitRegionTerminator()` (#69043) Fix for a crash reported in #64975. The crash occurs in the cast located [here](https://github.com/llvm/llvm-project/blob/ece5dd101c7e4dc2fd23428abd312f75fd3d3eaf/mlir/lib/Analysis/DataFlow/DeadCodeAnalysis.cpp#L262) because `llvm.unreachable` doesn't implement `RegionBranchTerminatorOpInterface`. The crash is caused by `DeadCodeAnalysis` assuming that `isa<RegionBranchOpInterface>(op->getParentOp())` implies `isa<RegionBranchTerminatorOpInterface>(op)` in `DeadCodeAnalysis::visit()`. This patch tried to fix this by enabling the analysis to proceed regardless of whether `op` is a `RegionBranchTerminatorOpInterface`.
-
Jonas Hahnfeld authored
The pass only requires that it can determine a uniquely identified target at some offsets. Multiple relocations at the same offset are fine otherwise and will be required when adding exception handling support for RISC-V.
-
Johannes Doerfert authored
The tests "unoptimized" version is still broken, disabled for now.
-
Mehdi Amini authored
-
Mehdi Amini authored
-
Mehdi Amini authored
-
Mehdi Amini authored
-
Mehdi Amini authored
-
Owen Pan authored
-
Johannes Doerfert authored
-
Kazu Hirata authored
-
Kazu Hirata authored
Identified with misc-include-cleaner.
-
Kazu Hirata authored
-
Kazu Hirata authored
Identified misc-include-cleaner.
-
Jon Roelofs authored
It is already present in the yaml, but missing from the printed diagnostics.
-
Kazu Hirata authored
-
Kazu Hirata authored
-
Kazu Hirata authored
-
Kazu Hirata authored
Identified with misc-include-cleaner.
-
Jacques Pienaar authored
-
SR_team authored
Shows align for records and fields declarations in hover information. Example: ```cpp struct A { char a; short b; }; ``` For this struct hover informations shows: ``` Size: 4 bytes, alignment 2 bytes ``` 
-
- Oct 22, 2023
-
-
Florian Hahn authored
The argument is only used in const contexts. Simplifies a follow-up diff.
-
Florian Hahn authored
Add simplification for redundant trunc(zext A) pairs. Generally apply a transform from D149903. Depends on D159200. Reviewed By: Ayal Differential Revision: https://reviews.llvm.org/D159202
-
LLVM GN Syncbot authored
-
Nikolas Klauser authored
Reviewed By: #libc, ldionne Spies: ldionne, libcxx-commits Differential Revision: https://reviews.llvm.org/D155330
-
Stanislav Mekhanoshin authored
-
Brad Smith authored
-
Kazu Hirata authored
These files satisfy all of the following: - misc-include-cleaner indicates that these files do not need Endian.h. - They do not mention "endian" anywhere. - They do not include any *.inc or *.def, which could need llvm::support::endian.
-
Madhur Amilkanthwar authored
This patch adds "nice-to-have" feature in lit. it prints the total number of discovered tests at the beginning. It is covenient to see the total number of tests and avoid scrolling up to the beginning of log. Further, this patch also prints %ge of tests. This patch fixes tests pointed by previous attempt of landing this patch. Reviewed By: RoboTux, jdenny-ornl Co-authored-by:Madhur A <madhura@nvidia.com>
-