- May 05, 2021
-
-
Nathan James authored
Fixes https://llvm.org/PR35099. I'm not sure if this decision was intentional but its definitely confusing for users. Reviewed By: MyDeveloperDay, HazardyKnusperkeks, curdeius Differential Revision: https://reviews.llvm.org/D101628
-
Fangrui Song authored
-
Leonard Chan authored
This implements the flag proposed in RFC http://lists.llvm.org/pipermail/cfe-dev/2020-August/066437.html. The goal is to add a way to override the default target C++ ABI through a compiler flag. This makes it easier to test and transition between different C++ ABIs through compile flags rather than build flags. In this patch: - Store -fc++-abi= in a LangOpt. This isn't stored in a CodeGenOpt because there are instances outside of codegen where Clang needs to know what the ABI is (particularly through ASTContext::createCXXABI), and we should be able to override the target default if the flag is provided at that point. - Expose the existing ABIs in TargetCXXABI as values that can be passed through this flag. - Create a .def file for these ABIs to make it easier to check flag values. - Add an error for diagnosing bad ABI flag values. Differential Revision: https://reviews.llvm.org/D85802
-
Matt Morehouse authored
It was previously disabled for the past 6+ months. I tried to re-enable it after some deflaking, but it still fails occasionally.
-
Nikita Popov authored
We need to use a logical or instead of a bitwise or to preserve poison behavior. Poison from the second condition should not propagate if the first condition is true. We were already handling this correctly in FoldBranchToCommonDest(), but not in this fold. (There are still other folds with this issue.)
-
Nikita Popov authored
Regenerate the branch weight test using --check-globals.
-
Nikita Popov authored
-
Matt Morehouse authored
Increase runs to 200,000 since we currently get a random failure about once per day on the buildbot.
-
Fangrui Song authored
Use the user-facing `-M no-aliases` instead.
-
Fangrui Song authored
--riscv-no-aliases is an internal cl::opt option not intended to be exported. Use the user-facing -M no-aliases instead.
-
Dávid Bolvanský authored
-
Andrew Savonichev authored
If a return value is explicitly rounded to 64 bits, an additional zext instruction is emitted, and in some cases it prevents tail call optimization. As discussed in D100225, this rounding is not necessary and can be disabled. Differential Revision: https://reviews.llvm.org/D100591
-
Arthur Eubanks authored
-
Nathan James authored
Change instances where options which are boolean are assigned the value 1|0 to use true|false instead. Reviewed By: aaron.ballman Differential Revision: https://reviews.llvm.org/D101721
-
Fangrui Song authored
-a is alias for --archive-headers, not --all-headers
-
Fraser Cormack authored
This patch adds the two MVTs to fix a legalizer crash when using vector shuffles of <256 x i16> and <128 x i16> on RISC-V. The legalizer can't promote the operand of `v256i32 = any_extend_vector_inreg v128i16`. Reviewed By: craig.topper, RKSimon Differential Revision: https://reviews.llvm.org/D101769
-
Ahsan Saghir authored
This patch prevents argument promotion of types having type size greater than 128 bits. Fixes Bugzilla: https://bugs.llvm.org/show_bug.cgi?id=49952 Reviewed By: #powerpc, nemanjai Differential Revision: https://reviews.llvm.org/D101188
-
Bjorn Pettersson authored
This reverts commit 3655f075. It caused assertion failures related to setLastUser in polly builds.
-
Louis Dionne authored
-
Wei Mi authored
Root node in ProfiledCallGraph. In ProfiledCallGraph::addProfiledFunction, to add a function symbol into the ProfiledCallGraph, currently an uninitialized ProfiledCallGraphNode node is created by ProfiledFunctions[Name] and inserted into Callees set of Root node before the node is initialized. The Callees set use ProfiledCallGraphNodeComparer as its comparator so the uninitialized ProfiledCallGraphNode may fail to be inserted into Callees set if it happens to contain a name in memory which has been inserted into the Callees set before. The problem will prevent some function symbols from being annotated with profiles and cause performance regression. The patch fixes the problem. Differential Revision: https://reviews.llvm.org/D101815
-
Fangrui Song authored
When dumping multiple pieces of information (e.g. --all-headers), there is sometimes no separator between two pieces. This patch uses the "\nheader:\n" style, which generally improves compatibility with GNU objdump. Note: objdump -t/-T does not add a newline before "SYMBOL TABLE:" and "DYNAMIC SYMBOL TABLE:". We add a newline to be consistent with other information. `objdump -d` prints two empty lines before the first 'Disassembly of section'. We print just one with this patch. Differential Revision: https://reviews.llvm.org/D101796
-
gbreynoo authored
Reapply 73686247 after revert and fix Looking at other tools using tablegen for help output, general options like --help are not separated from other options. This change removes the "Generic Options" option group so the options are listed together. the macho specific option group is left unaffected. The test help.test was modified to reflect this change. Differential Revision: https://reviews.llvm.org/D101652
-
Med Ismail Bennani authored
A source location specifier class that holds a Declaration object containing a FileSpec with line and column information. The column line is optional. It also holds search flags that can be fetched by resolvers to look inlined declarations and/or exact matches. It describes a specific location in a source file and allows the user to perform checks and comparaisons between multiple instances of that class. Differential Revision: https://reviews.llvm.org/D100962 Signed-off-by:
Med Ismail Bennani <medismail.bennani@gmail.com>
-
Med Ismail Bennani authored
This patch moves the Declaration class from the Symbol library to the Core library. This will allow to use it in a more generic fashion and aims to lower the dependency cycles when it comes to the linking. The patch also does some cleaning up by making column information permanent and removing the LLDB_ENABLE_DECLARATION_COLUMNS directives. Differential revision: https://reviews.llvm.org/D101556 Signed-off-by:
Med Ismail Bennani <medismail.bennani@gmail.com>
-
Med Ismail Bennani authored
Update FileSpec doxygen path to reflect its actual location in the source-tree. Signed-off-by:Med Ismail Bennani <medismail.bennani@gmail.com>
-
Dimitry Andric authored
This reverts commit 73686247, as there were git stash conflict markers left unresolved.
-
Christudasan Devadasan authored
Removing an assertion introduced with D68945. The patch was later reverted with 6531a78a, but failed to remove this assertion. It causes a problem while trying to split a 64-bit argument into sub registers. Reviewed By: arsenm Differential Revision: https://reviews.llvm.org/D101594
-
Dimitry Andric authored
This fixes PR49821, and avoids "ld.lld: error: test.o:(.rodata.str1.1): offset is outside the section" errors when linking MIPS objects with negative R_MIPS_LO16 implicit addends. ld.lld handles R_MIPS_HI16/R_MIPS_LO16 separately, not as a whole, so it doesn't know that an R_MIPS_HI16 with implicit addend 1 and an R_MIPS_LO16 with implicit addend -32768 represents 32768, which is in range of a MergeInputSection. We could introduce a new RelExpr member (like R_RISCV_PC_INDIRECT for R_RISCV_PCREL_HI20 / R_RISCV_PCREL_LO12) but the complexity is unnecessary given that GNU as keeps the original symbol for this case as well. Adds a new test case for PR49821, and also updates two other test cases that are affected by this change. Reviewed By: atanasyan, MaskRay Differential Revision: https://reviews.llvm.org/D101773
-
Jennifer Yu authored
as used. The problem only happens with constexpr variable, for constexpr variable, variable is not marked during parser variable. This is because compiler might find some var's associate expressions may not actully an odr-used later, the variables get kept in MaybeODRUseExprs, in normal case, at end of process fullExpr, the variable will be marked during the call to CleanupVarDeclMarking(). Since we are processing expression of OpenMP clauses, and the ActOnFinishFullExpr is not getting called that casue variable is not get marked. One way to fix this is to call CleanupVarDeclMarking() in EndOpenMPClause for each omp directive. This to fix https://bugs.llvm.org/show_bug.cgi?id=50206 Differential Revision: https://reviews.llvm.org/D101781
-
Fabian Meumertzheim authored
Currently, the position hint of an entry in the persistent auto dictionary is fixed to 1. As a consequence, with a 50% chance, the entry is applied right after the first byte of the input. As the position 1 does not appear to have any particular significance, this is likely a bug that may have been caused by confusing the constructor parameter with a success count. This commit resolves the issue by preserving any existing position hint or disabling the hint if the original entry didn't have one. Reviewed By: morehouse Differential Revision: https://reviews.llvm.org/D101686
-
- May 04, 2021
-
-
Sanjay Patel authored
This fixes https://llvm.org/PR48900 , but as seen in the regression tests prevents some optimizations. There are a few options to restore those (switch to min/max intrinsics, add larger pattern matching for select with dominating condition, improve CVP), but we need to prevent the bug 1st.
-
gbreynoo authored
Looking at other tools using tablegen for help output, general options like --help are not separated from other options. This change removes the "Generic Options" option group so the options are listed together. the macho specific option group is left unaffected. The test help.test was modified to reflect this change. Differential Revision: https://reviews.llvm.org/D101652
-
Amy Kwan authored
This patch updates the scalar atomic patterns to use the refactored load/store implementation introduced in D93370. All existing test cases pass with when the refactored patterns are utilized. Differential Revision: https://reviews.llvm.org/D94498
-
William S. Moses authored
See: https://reviews.llvm.org/D101710
-
gbreynoo authored
The llvm-objdump command guide has the option --cfg which was removed from the tool by 888320e9 in 2014. This change updates the command guide to reflect this. Differential Revision: https://reviews.llvm.org/D101648
-
William S. Moses authored
Differential Revision: https://reviews.llvm.org/D101801
-
Florian Hahn authored
This patch updates the code handling reduction recipes to also keep track of the incoming value from the latch in the recipe. This is needed to model the def-use chains completely in VPlan, so that it is possible to replace the incoming value with an arbitrary VPValue. Reviewed By: Ayal Differential Revision: https://reviews.llvm.org/D99294
-
William S. Moses authored
Differential Revision: https://reviews.llvm.org/D101710
-
Zachary Henkel authored
The CALLBACK macro is used extensively in the Windows SDK.
-
Louis Dionne authored
It's not a default configuration, so it belongs to stage 3.
-