- Feb 28, 2020
-
-
Fangrui Song authored
-
- Feb 27, 2020
-
-
Sven van Haastregt authored
Differential Revision: https://reviews.llvm.org/D75125
-
Sven van Haastregt authored
-
Peter Wu authored
Differential revision: https://reviews.llvm.org/D75012
-
Hans Wennborg authored
This caused PR44953. See also the discussion on D74846. This reverts commit cbc9d22e. (cherry picked from commit 7ea9a6e0)
-
Sam Elliott authored
This corrects some typos and clarifies some points.
-
Hans Wennborg authored
The code in llvmorg-10-init-12188-g25ce33a6 is a breaking change for users of older linkers who don't pass a version parameter, which prevents a drop-in clang upgrade. Old tools can't know about what future tools will do, so as a general principle the burden should be new tools to be compatible by default. Also, for comparison, none of the other tests of Version within AddLinkArgs add any new behaviors unless the version is explicitly specified. Therefore, this patch changes the -platform_version behavior from opt-out to opt-in. Patch by David Major! Differential revision: https://reviews.llvm.org/D74784 (cherry picked from commit 5122e828)
-
Roman Lebedev authored
Much like with reassociateShiftAmtsOfTwoSameDirectionShifts(), as input, we have the following pattern: icmp eq/ne (and ((x shift Q), (y oppositeshift K))), 0 We want to rewrite that as: icmp eq/ne (and (x shift (Q+K)), y), 0 iff (Q+K) u< bitwidth(x) While we know that originally (Q+K) would not overflow (because 2 * (N-1) u<= iN -1), we may have looked past extensions of shift amounts. so it may now overflow in smaller bitwidth. To ensure that does not happen, we need to ensure that the total maximal shift amount is still representable in that smaller bitwidth. If the overflow would happen, (Q+K) u< bitwidth(x) check would be bogus. https://bugs.llvm.org/show_bug.cgi?id=44802 (cherry picked from commit 2855c8fe)
-
Roman Lebedev authored
https://bugs.llvm.org/show_bug.cgi?id=44802 (cherry picked from commit 6f807ca0)
-
Roman Lebedev authored
As input, we have the following pattern: Sh0 (Sh1 X, Q), K We want to rewrite that as: Sh x, (Q+K) iff (Q+K) u< bitwidth(x) While we know that originally (Q+K) would not overflow (because 2 * (N-1) u<= iN -1), we may have looked past extensions of shift amounts. so it may now overflow in smaller bitwidth. To ensure that does not happen, we need to ensure that the total maximal shift amount is still representable in that smaller bitwidth. If the overflow would happen, (Q+K) u< bitwidth(x) check would be bogus. https://bugs.llvm.org/show_bug.cgi?id=44802 (cherry picked from commit 781d077a)
-
Roman Lebedev authored
https://bugs.llvm.org/show_bug.cgi?id=44802 (cherry picked from commit 425ef999)
-
Juneyoung Lee authored
Reviewers: hans, nlopes, regehr Subscribers: llvm-commits Tags: #llvm Differential Revision: https://reviews.llvm.org/D75226
-
Nathan James authored
Summary: Fixes [[ https://bugs.llvm.org/show_bug.cgi?id=42193 | hasName AST matcher is confused by extern "C" in namespace. ]] Reviewers: klimek, aaron.ballman, gribozavr2 Reviewed By: aaron.ballman Subscribers: cfe-commits Tags: #clang Differential Revision: https://reviews.llvm.org/D75202 (cherry picked from commit 16cabf27)
-
Hans Wennborg authored
MC currently does not emit these relocation types, and lld does not handle them. Add FKF_Constant as a work-around of some ARM code after D72197. Eventually we probably should implement these relocation types. By Fangrui Song! Differential revision: https://reviews.llvm.org/D72892 (cherry picked from commit 2e24219d)
-
Nikita Popov authored
MemorySSA is often taking up an unreasonable fraction of runtime in assertion enabled builds. Turns out that there is one code-path that runs verifyMemorySSA() even if VerifyMemorySSA is not enabled. This patch makes it conditional as well. Differential Revision: https://reviews.llvm.org/D74505 (cherry picked from commit f0b57d80)
-
Hans Wennborg authored
It no longer applies after d7afdb59. This reverts commit 002af011.
-
Michael Kruse authored
Pass plugins introduced in D61446 do not support dynamic linking on Windows, hence the option LLVM_${name_upper}_LINK_INTO_TOOLS can only work being set to "ON". Currently, it defaults to "OFF" such that such plugins are inoperable by default on Windows. Change the default for subprojects to follow LLVM_ENABLE_PROJECTS. Reviewed By: serge-sans-paille, MaskRay Differential Revision: https://reviews.llvm.org/D72372 (cherry picked from commit 6369b9bf) This is for PR45001. -
Sam Elliott authored
-
Florian Hahn authored
Reviewers: anemet, Gerolf Reviewed By: anemet Differential Revision: https://reviews.llvm.org/D75161
-
- Feb 26, 2020
-
-
Philip Reames authored
This reverts commit 8d22100f. There was a functional regression reported (https://bugs.llvm.org/show_bug.cgi?id=44996). I'm not actually sure the patch is wrong, but I don't have time to investigate currently, and this line of work isn't something I'm likely to get back to quickly. (cherry picked from commit 14845b2c)
-
Hans Wennborg authored
Differential revision: https://reviews.llvm.org/D75121 (cherry picked from commit 41a6612e)
-
Roman Lebedev authored
This reverts https://reviews.llvm.org/D58468 (rL354676, 44037d7a), and all and any follow-ups to that code block. https://bugs.llvm.org/show_bug.cgi?id=43446 (cherry picked from commit d20907d1)
-
Hans Wennborg authored
See the discussion on PR44792. This reverts commit 02ce9d8e.
-
Rong Xu authored
This patch fixes PR44896. For IR input files, option fdiscard-value-names should be ignored as we need named values in loadModule(). Commit 60d39479 sets this option after loadModule() where valued names already created. This creates an inconsistent state in setNameImpl() that leads to a seg fault. This patch forces fdiscard-value-names to be false for IR input files. This patch also emits a warning of "ignoring -fdiscard-value-names" if option fdiscard-value-names is explictly enabled in the commandline for IR input files. Differential Revision: https://reviews.llvm.org/D74878 (cherry picked from commit 11857d49)
-
Alina Sbirlea authored
Summary: Potential fix for: https://bugs.llvm.org/show_bug.cgi?id=44889 and https://bugs.llvm.org/show_bug.cgi?id=44408 In the legacy pass manager, loop rotate need not compute MemorySSA when not being in the same loop pass manager with other loop passes. There isn't currently a way to differentiate between the two cases, so this attempts to limit the usage in LoopRotate to only update MemorySSA when the analysis is already available. The side-effect of this is that it will split the Loop pipeline. This issue does not apply to the new pass manager, where we have a flag specifying if all loop passes in that loop pass manager preserve MemorySSA. Reviewers: dmgreen, fedor.sergeev, nikic Subscribers: Prazek, hiraditya, george.burgess.iv, llvm-commits Tags: #llvm Differential Revision: https://reviews.llvm.org/D74574 (cherry picked from commit 1326a5a4)
-
Kadir Cetinkaya authored
Summary: Fixes https://github.com/clangd/clangd/issues/279. We were removing the color options but not the preceeding -Xclang which causes errors since the -Xclang would now apply to the next option in the list of options. Now, when removing a color option, we check if there was a preceeding -Xclang and remove it as well. Patch By @DaanDeMeyer ! Reviewers: sammccall, kadircet Reviewed By: sammccall Subscribers: ilya-biryukov, usaxena95 Differential Revision: https://reviews.llvm.org/D75019 (cherry picked from commit da236f23)
-
Michael Kruse authored
In release 10.0, Polly is not linked into opt/bugpoint/clang by default anymore. Add workarounds in release notes.
-
Dan Gohman authored
-
- Feb 25, 2020
-
-
Sam McCall authored
-
Hans Wennborg authored
No functionality change (intended), but this seems to make the code a bit clearer for the compiler and maybe for human readers too. (cherry picked from commit edae4be8)
-
Hans Wennborg authored
Now that the Windows installer no longer does anything besides self-extract, maybe it would make sense to distribute the toolchain as a plain zip file in addition to the current installer. Differential revision: https://reviews.llvm.org/D74896 (cherry picked from commit 4486aa03)
-
Hans Wennborg authored
By Andrew Paverd! Differential revision: https://reviews.llvm.org/D75047
-
Hans Wennborg authored
The previous patch (cff90f07) didn't cover ARM. (cherry picked from commit decd021f)
-
Hans Wennborg authored
See https://github.com/llvm/llvm-project/issues/141 (cherry picked from commit dcd89b3d)
-
Amy Huang authored
-
Pavel Labath authored
Mainly involves location and range list handling, but other holes have been filled too.
-
- Feb 24, 2020
-
-
Hans Wennborg authored
By Kristof Beyls!
-
Hans Wennborg authored
By Johannes Doerfert!
-
Hans Wennborg authored
By Alexey Bataev!
-
- Feb 22, 2020
-
-
Tyker authored
-