Merge feature branch to the bump branch #44
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Update the `actions/download-artifact` action to version 3. - Use the major version of an action instead of specifying the minor or patch versions. Differential Revision: https://reviews.llvm.org/D148327
Avoids some regressions when the implementation of isKnownNeverNaN is replaced with computeKnownFPClass.
Limit turns out to be implemented in the exact same way for all calls to tryToVectorizeSequence(). So this patch removes it and implements it internally as a lambda function. Differential Revision: https://reviews.llvm.org/D148382
…o compare unnamed bit-fields If we look at class.bit p2 it tells us that that unnamed bit-fields are not members and class.compare.default p5 tells us that we should only compare non-static data members of the class. This fixes: llvm#61335 and llvm#61417 Differential Revision: https://reviews.llvm.org/D146329
…for Fuchsia"" This reverts commit 39ece75. Reverting because I suspect this is leading to the failures we see in fxbug.dev/125426. We're seeing unexpected OOMs and I suspect it's because this change makes c-style allocation functions use hwasan's allocator which might be worse than the default one (scudo). Reverting for now to see if this unblocks the clang roll.
Towards converting our use of llvm::StringView to std::string_view, remove a method that std::string_view doesn't have. This could be moved to the nascent llvm/ADT/StringViewExtras.h, but the use is highly localized to one TU. Move this to be a static function there. Reviewed By: MaskRay Differential Revision: https://reviews.llvm.org/D148375
added getchar and getchar_unlocked which are just wrappers getc and getc_unlocked respectively. Reviewed By: sivachandra, lntue, michaelrj Differential Revision: https://reviews.llvm.org/D147919
This refactoring was waiting on converting LLVM to C++17. Leave StringView.h and cleanup around for subsequent cleanup. Reviewed By: MaskRay Differential Revision: https://reviews.llvm.org/D148384
…urrent target Catch accidentally used flags and match MCSectionELF.cpp `MCSectionELF::printSwitchToSection`. Reviewed By: aeubanks Differential Revision: https://reviews.llvm.org/D148386
The explicit `ignoring all compiler directives` reminder warning is no longer accurate. Any similar, more accurate message is best generated by the front end (change pending).
PATCHABLE_* instructions expand to up to 36-byte sleds. Updating the size of PATCHABLE instructions causes them to be outlined, so we need to add a check to prevent the outliner from considering basic blocks that contain PATCHABLE instructions. Differential Revision: https://reviews.llvm.org/D147982
Differential Revision: https://reviews.llvm.org/D147896
…`shl` can be zero.
Previously only return `shl` non-zero if the shift value was `1`. We
can expand this if we have some bounds on the shift count.
For example:
```
%cnt = and %c, 16 ; Max cnt == 16
%val = or %v, 4 ; val[2] is known one
%shl = shl %val, %cnt ; (val.known.one << cnt.maxval) != 0
```
Differential Revision: https://reviews.llvm.org/D147897
Alive2 Link: https://alive2.llvm.org/ce/z/mxZLJn Reviewed By: nikic Differential Revision: https://reviews.llvm.org/D147898
Fixes diagnostics reported against https://reviews.llvm.org/D148384 https://lab.llvm.org/buildbot/#/builders/127/builds/46749/steps/4/logs/stdio Reviewed By: MaskRay Differential Revision: https://reviews.llvm.org/D148392
Future patch will set the current context as well. Existing callsite requires additional lock to find context. Differential Revision: https://reviews.llvm.org/D148390
After enabling ArgPromotion at lower opt levels, the body of this test was optimized away, and the CHECK lines no longer functioned. Setting -O0 is a hammer, but gets the test passing again, until code owners can find a better solution. Reviewed By: fmayer Differential Revision: https://reviews.llvm.org/D148396
Reviewed By: MaskRay Differential Revision: https://reviews.llvm.org/D148358
PrintIRInstrumentation::shouldPrintAfterPass accepts a pass ID instead of a pass name Reviewed By: aeubanks Differential Revision: https://reviews.llvm.org/D147394
…ension section. NFC
…ain. NFC Now that we use IntrusiveRefCntPtr instead of std::shared_ptr we don't need to pass a TreePatternNodePtr to this method.
…level. This patch refines the matching of the relevant optional types to anchor on the global namespace. Previously, we could match anything with the right name (e.g. `base::Optional`) even if nested within other namespaces. This over matching resulted in an assertion violation when _different_ `base::Optional` was encountered nested inside another namespace. Fixes issue llvm#57036. Differential Revision: https://reviews.llvm.org/D148344
Adjusts the matchers in the optional model to avoid dependency on internal implementation details of libc++'s `std::optional`. In the process, factors out the code to check the name of these types so that it's shared throughout. Differential Revision: https://reviews.llvm.org/D148377
Add MSP430 to the list of available targets, implement MSP430 ABI, add support for debugging targets with 16-bit address size. The update is intended for use with MSPDebug, a GDB server implementation for MSP430. Reviewed By: bulbazord, DavidSpickett Differential Revision: https://reviews.llvm.org/D146965
This reverts commit 9395cf0 because it breaks the modules build: STLFunctionalExtras.h:54:3: error: 'llvm::function_ref<std::__1::optional<std::__1::basic_string<char>> (llvm::StringRef, llvm::StringRef)>::function_ref' from module 'LLVM_Utils.ADT.STLFunctionalExtras' is not present in definition of 'llvm::function_ref<std::__1::optional<std::__1::basic_string<char>> (llvm::StringRef, llvm::StringRef)>' in module 'LLVM_Utils.ADT.STLFunctionalExtras' https://green.lab.llvm.org/green/view/LLDB/job/lldb-cmake/53806/
We need to create a new type with transposed shape after transposing the operand in `CanonicalizeContractMatmulToMMT`. Reviewed By: kuhar, dcaballe Differential Revision: https://reviews.llvm.org/D148470
This reverts commit 82c02b7. Apparently, the original patch was not rebased onto `main
…` and `[` as `is` is a keyword in C# ensure there is always a space before the `[` regardless of `SpaceBeforeSquareBrackets` setting Fixes: llvm#61965 llvm#61965 Reviewed By: owenpan, HazardyKnusperkeks Differential Revision: https://reviews.llvm.org/D148472
…sting functions. This patch replaces the uses of PointerUnion.is function by llvm::isa, PointerUnion.get function by llvm::cast, and PointerUnion.dyn_cast by llvm::dyn_cast_if_present. This is according to the FIXME in the definition of the class PointerUnion. This patch does not remove them as they are being used in other subprojects. Reviewed By: mehdi_amini Differential Revision: https://reviews.llvm.org/D148449
This allows the subsequent code to access the E_IDENT fields, which will be used in the upcoming JITLink ppc64 backend (https://reviews.llvm.org/D148192).
- Separate the two test and only have TestSymbolFileJSON rely on strip. - Use different file names to make sure LLDB reloads the module. This should address all the post commit review from D148062.
Reviewed By: kwk Differential Revision: https://reviews.llvm.org/D143535
This patch updates the list of supported libs according to the latest changes to x86_64 Reviewed By: sivachandra Differential Revision: https://reviews.llvm.org/D147961
…n VPlan." This reverts commit 7fc0b30. Causes a clang hang when building xz utils, github issue llvm#62187.
The `__builtin_bswap{32,64}()` builtins (introduced in commit e07a421)
are missing from MSVC, which causes build errors when compiling LLDB on
Windows (tested with MSVC 19.34.31943.0). This patch replaces the
builtins with either MSVC's `_byteswap_u{long,64}()` or the original
builtins, or the `bswap_{32,64}()` functions from byteswap.h, depending
on which ones are available.
Reviewed By: bulbazord
Differential Revision: https://reviews.llvm.org/D148541
Use the assembly format with custom parser/printer for specific clauses instead of a full custom parser/printer. Reviewed By: PeteSteinfeld Differential Revision: https://reviews.llvm.org/D148391
Partial revert of commit ee91799. I can't reproduce the error locally, so let's revert it to keep the bots happy.
Ferdinand.llvm bump may 0305
flemairen6
changed the base branch from
max.FXML-1406.bump_llvm_to_green_commit
to
feature/fused-ops
flemairen6
changed the base branch from
feature/fused-ops
to
max.FXML-1406.bump_llvm_to_green_commit
mgehre-amd
approved these changes
Jun 15, 2023
mgehre-amd
changed the base branch from
max.FXML-1406.bump_llvm_to_green_commit
to
feature/fused-ops
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.