- Dec 19, 2021
-
-
Krasimir Georgiev authored
It appears that this regressed the formatting of initializer lists in some cases, see comments on https://reviews.llvm.org/D114583. I'll follow-up by adding regression tests for these. This reverts commit c41b3b0f. Reviewed By: MyDeveloperDay Differential Revision: https://reviews.llvm.org/D116000
-
Christian Ulmann authored
This commit rewrites most existing unittests involving FlatAffineConstraints to use the parsing utility. This helps to make the tests more understandable. Reviewed By: arjunp Differential Revision: https://reviews.llvm.org/D115920
-
Simon Pilgrim authored
Match order of most of the other integer opcode combines
-
Simon Pilgrim authored
The IceLake scheduler model is still mainly a copy of the SkylakeServer model. This patch adjusts the fp shuffle classes to account for most instructions now working on Port 1 as well as Port 5. This is based off Agner + uops.info as well as the PR48110 report. Differential Revision: https://reviews.llvm.org/D115752
-
Simon Pilgrim authored
Merge the node combines into a common DAGCombiner::visitFMinMax (like we do for IMINMAX). Move the constant folding into SelectionDAG::foldConstantFPMath. This allows us to fold the vecreduce-propagate-sd-flags.ll test as it reduces constants - so I've refactored it to take variables instead. Differential Revision: https://reviews.llvm.org/D115952
-
Shilei Tian authored
-
Cyndy Ishida authored
-
Craig Topper authored
The tests no longer generate the instructions that are mentioned in the comments.
-
Jakub Kuderski authored
Reviewed By: arsenm Differential Revision: https://reviews.llvm.org/D115900
-
Jakub Kuderski authored
The race condition happened when two pass managers ran on two different modules but modified/read the global variables. To address this, I considered using singletons and freestanding functions to allow getting/setting `HardwareLimits` and `RegisterEncoding`, or making it local to the pass. I chose the latter and made it a member of `WaitcntsBrackets`, to minimizes the amount of global state. Reviewed By: arsenm Differential Revision: https://reviews.llvm.org/D115896
-
Craig Topper authored
Drop stale FIXME.
-
Michael Berg authored
Both these preference helper functions have initial support with this change. The loop unrolling preferences are set with initial settings to control thresholds, size and attributes of loops to unroll with some tuning done. The peeling preferences may need some tuning as well as the initial support looks much like what other architectures utilize. Reviewed By: craig.topper Differential Revision: https://reviews.llvm.org/D113798
-
Mehdi Amini authored
-
Nikita Popov authored
This reverts commit 9fd4f80e. This breaks SingleSource/Regression/C/gcc-c-torture/execute/pr19687.c in test-suite. Either the test is incorrect, or clang is generating incorrect union initialization code. I've submitted https://reviews.llvm.org/D115994 to fix the test, assuming my interpretation is correct. Reverting this in the meantime as it may take some time to resolve.
-
Martin Storsjö authored
When run in a git bash terminal, sys.stderr isn't flushed implicitly after printing each line. Manually flush it after each printout, to avoid getting broken/misordered output. A similar fix had been done in the old libcxx test config, committed as part of 7e3ee09a / D28725; this generalizes the fix, making it available in the new libcxx test configs too, and for any other test that uses lit_config.note(). Differential Revision: https://reviews.llvm.org/D115761
-
Cyndy Ishida authored
For scripting purposes, use different error code for cases that aren't a result of different tbd files, e.g. bad input like a path to a broken symlink. This behaves more similiarly to how the unix `diff` command behaves. This also includes minor tweaks on error messages. Reviewed By: ributzka, JDevlieghere Differential Revision: https://reviews.llvm.org/D115905
-
Ricky Zhou authored
Before this change, AAResults::getModRefInfo() was missing a case for callbr instructions (asm goto), which may read/write memory. In PR52735, this led to a miscompile where a load was incorrect eliminated. Add this missing case, as well as an assert verifying that all memory-accessing instructions are handled properly. Fixes #52735. Differential Revision: https://reviews.llvm.org/D115992
-
Nikita Popov authored
This was creating an AttributeSet from an AttrBuilder, and then (implicitly) constructing an AttrBuilder from the AttributeSet, and then the method would internally convert that back into an AttributeSet. Instead, directly pass the AttrBuilder.
-
Matt Arsenault authored
-
- Dec 18, 2021
-
-
Matt Arsenault authored
-
mydeveloperday authored
https://github.com/llvm/llvm-project/issues/49804 Interaction between IndentExternBlock and AfterExternBlock means you cannot have AfterExternBlock = true and IndentExternBlock = NoIndent/Indent This patch resolves that ``` BraceWrapping: AfterExternBlock: true IndentExternBlock: AfterExternBlock ``` Fixes: #49804 Reviewed By: HazardyKnusperkeks, curdeius, owenpan Differential Revision: https://reviews.llvm.org/D115879
-
Konstantin Boyarinov authored
Minor change - add tests that std::array methods (data, (c/r)begin, (c/r)end) are noexcept Reviewed By: ldionne, rarutyun, #libc Differential Revision: https://reviews.llvm.org/D115717
-
mydeveloperday authored
https://github.com/llvm/llvm-project/issues/43196 Fixes #43196 -> is incorrectly interpreted as a TrailingReturnArrow if we've seen an auto ``` auto p = new A; auto x = p -> foo<1>(); ``` Reviewed By: curdeius Differential Revision: https://reviews.llvm.org/D115903
-
Nikita Popov authored
This reverts commit a8a51fe5. This breaks the strncpy-overflow.cpp test case.
-
Fangrui Song authored
With this patch, writing .debug_str is significantly for a program with 1.5G .debug_str: * .debug_info 1.22s * .debug_str 2.57s decreases to 0.66
-
Kazu Hirata authored
-
Fangrui Song authored
This decreases struct sizes and usually decreases the lld executable size (39KiB for my x86-64 executable) (unless in some cases smaller SmallVector leads to more inlining, e.g. StringTableBuilder). For --gdb-index, there may be memory usage saving.
-
Joshua Huels authored
This fixes the regex in ClangFormatStyleOptions for IncludeCategories to match anything starting with < or starting with | AND followed by (gtest|gmock|isl|json) then /. Differential Revision: https://reviews.llvm.org/D115910
-
Kazu Hirata authored
-
Jonas Devlieghere authored
This is part of a bigger rework of the reproducer feature. See [1] for more details. [1] https://lists.llvm.org/pipermail/lldb-dev/2021-September/017045.html
-
Kazu Hirata authored
-
Yuanfang Chen authored
if the pascal case ones failed. Some tools, like msvc-wine/xwin, use the lower case.
-
Julian Lettner authored
With the introduction of Apple Silicon `defined(__aarch64__)` is not a reliable way to check for the platform anymore. We want to use the "normal" `Mapping48AddressSpace` mapping everywhere except devices, including the iOS simulators on AS. Relevant revisions: https://reviews.llvm.org/D35147 https://reviews.llvm.org/D86377 https://reviews.llvm.org/D107743 https://reviews.llvm.org/D107888 Differential Revision: https://reviews.llvm.org/D115843
-
Julian Lettner authored
Previously we would crash in the TSan runtime if the user program passes a pointer to `malloc_size()` that doesn't point into app memory. In these cases, `malloc_size()` should return 0. For ASan, we fixed a similar issue here: https://reviews.llvm.org/D15008 Radar-Id: rdar://problem/86213149 Differential Revision: https://reviews.llvm.org/D115947
-
Vitaly Buka authored
It was added D115852 and related patches and it brakes https://lab.llvm.org/buildbot/#/builders/fuchsia-x86_64-linux
-
Kazu Hirata authored
-
Jon Chesterfield authored
-
Ellis Hoag authored
There was a build failure on the `instrprof-debug-info-correlate.c` test because zlib was missing so we need to require it to run the test. Differential Revision: https://reviews.llvm.org/D115970
-
Marek Kurdej authored
Reviewed By: MyDeveloperDay Differential Revision: https://reviews.llvm.org/D115968
-