- Oct 20, 2022
-
-
Eli Friedman authored
The way this code checks whether a pointer is null is wrong for other reasons; it doesn't actually check whether a null pointer constant is a "constant" in the C++ standard sense. But this fix at least makes sure we don't treat a non-null pointer as if it were null. Fixes https://github.com/llvm/llvm-project/issues/57883 Differential Revision: https://reviews.llvm.org/D134928
-
Krzysztof Parzyszek authored
HVC::calculatePointerDifference inserts temporary instructions for simplification, and calulation of known bits. These instructions were inserted at the end of a basic block (after the terminator), which caused BB->getTerminator() to return nullptr. This, in turn, caused a crash when a PHI instruction was examined in computeKnownBits.
-
wren romano authored
Per https://reviews.llvm.org/D136005#3866692 the introduction of the MLIRSparseTensorEnums target in D136002 caused breakage on some versions of cmake. This differential aims to fix those errors. Reviewed By: aartbik Differential Revision: https://reviews.llvm.org/D136217
-
Augusto Noronha authored
Currently, SymbolFileDWARFDebugMap works on the assumption that there is only one compile unit per object file. This patch documents this limitation (when using the general SymbolFile API), and allows users of the concrete SymbolFileDWARFDebugMap class to find out about these extra compile units. Differential Revision: https://reviews.llvm.org/D136114
-
Mircea Trofin authored
The test was checking output opcodes, one changed as result of D135934.
-
Thomas Raoux authored
The transition to transform dialect based tests dropped several cases of the split reduction testing. Adding them back. Differential Revision: https://reviews.llvm.org/D136287
-
Teresa Johnson authored
Extracted from D135714 which adds summary support for MemProf. We will need a 3rd tuple member in the ValueIdToValueInfoMap, this patch makes a number of NFC changes to the existing clients of that map to reflect the conversion of pair to tuple.
-
Michal Paszkowski authored
Differential Revision: https://reviews.llvm.org/D135904
-
Michal Paszkowski authored
Differential Revision: https://reviews.llvm.org/D135902
-
Jorge Gorbe Moya authored
This patch adds a new matching method for data formatters, in addition to the existing exact typename and regex-based matching. The new method allows users to specify the name of a Python callback function that takes a `SBType` object and decides whether the type is a match or not. Here is an overview of the changes performed: - Add a new `eFormatterMatchCallback` matching type, and logic to handle it in `TypeMatcher` and `SBTypeNameSpecifier`. - Extend `FormattersMatchCandidate` instances with a pointer to the current `ScriptInterpreter` and the `TypeImpl` corresponding to the candidate type, so we can run registered callbacks and pass the type to them. All matcher search functions now receive a `FormattersMatchCandidate` instead of a type name. - Add some glue code to ScriptInterpreterPython and the SWIG bindings to allow calling a formatter matching callback. Most of this code is modeled after the equivalent code for w...
-
Alexey Bataev authored
-
Jolanta Jensen authored
Differential Revision: https://reviews.llvm.org/D136241
-
Aart Bik authored
Reviewed By: Peiming Differential Revision: https://reviews.llvm.org/D136275
-
Andreas Hollandt authored
Fills gaps in the time trace when precompiled headers are created/loaded. Reviewed By: jansvoboda11 Differential Revision: https://reviews.llvm.org/D135657
-
Craig Topper authored
The existing prefix conflicted and the script silently dropped the checks.
-
Rafael Auler authored
We noticed some binaries with duplicated global symbol entries (same name, address and size). Ignore them as it is possibly a bug in the linker, and continue processing, unless the symbol has a different size or address. Reviewed By: #bolt, maksfb Differential Revision: https://reviews.llvm.org/D136122
-
Valentin Clement authored
D136189 was missing a test where the pass object is not at position 0. This patch adds one. Reviewed By: jeanPerier, PeteSteinfeld Differential Revision: https://reviews.llvm.org/D136231
-
Manish Gupta authored
-
Quentin Colombet authored
Add an entry for my office hours. Intended focus is low-level LLVM stuff. Differential Version: https://reviews.llvm.org/D136270
-
Fangrui Song authored
This reverts commit 8ef3fd8d. I mentioned that GlobalAlias was not handled. It turns out GlobalAlias has to be handled in the same patch (as opposed to in a follow-up), as otherwise clang codegen of C5/D5 constructor/destructor would regress (https://reviews.llvm.org/D135427#3869003).
-
Peiming Liu authored
Sparse compiler used to generate vectorized code for sparse tensors computation, but it should really be delegated to other vectorization passes for better progressive lowering. https://discourse.llvm.org/t/rfc-structured-codegen-beyond-rectangular-arrays/64707 Reviewed By: aartbik Differential Revision: https://reviews.llvm.org/D136183
-
Yuanfang Chen authored
The error is ``` NotADirectoryError: [Errno 20] Not a directory: '<build-dir>/unittests/Analysis/./AnalysisTests/0/40' ``` Exclude unittests when collecting features because unittests don't make use of feature keywords.
-
Vy Nguyen authored
- remove unused/duplicate includes - reformatting/whitespaces Differential Revision: https://reviews.llvm.org/D136266
-
Yuanfang Chen authored
This gives linker scripts a hint about where to place the section.
-
Alexander Yermolovich authored
GCC 12 produces DW_FORM_addr for DW_AT_call_return_pc. Added support for that. Fixes facebookincubator/BOLT#307 Reviewed By: maksfb Differential Revision: https://reviews.llvm.org/D136204
-
Chris Bieneman authored
This slipped in by accident.
-
Mark de Wever authored
Addresses post-commit review comment in D134742.
-
bixia1 authored
This is to allow the use of a nop convert to express that the sparse tensor allocated through bufferization::AllocTensorOp will be expanded to sparse tensor storage by sparse tensor codegen. Reviewed By: aartbik Differential Revision: https://reviews.llvm.org/D136214
-
Prabhdeep Singh Soni authored
This patch adds support for the `depend` clause for the `task` construct. Reviewed By: jdoerfert Differential Revision: https://reviews.llvm.org/D135695
-
Chris Bieneman authored
The ShaderFlagsAnalysisWrapper needs to be marked to preserve all analyssis. Fixes #58474 (https://github.com/llvm/llvm-project/issues/58474)
-
Sander de Smalen authored
The Key for the SubtargetMap had the StreamingSVEModeDisabled in the wrong place. This change is non-functional, since the string (key) is still unique.
-
Chris Bieneman authored
The pretty-printer isn't working because the resource analysis isn't properly preservered.
-
Caroline Concatto authored
This patch adds the assembly/disassembly for the following instructions: For INT: ADD(array results, multiple and single vector): Add replicated single vector to multi-vector with ZA array vector results. SUB(array results, multiple and single vector): Subtract replicated single vector from multi-vector with ZA array vector results. For FP: FMLA (multiple and single vector): Multi-vector floating-point fused multiply-add by vector. FMLS (multiple and single vector): Multi-vector floating-point multiply-subtract long by vector. The reference can be found here: https://developer.arm.com/documentation/ddi0602/2022-09 The Matriz Operand has 2 new sizes 32(.s) and 64(.d) bits (MatrixOp32 and MatrixOp64) Depends on: D135448 Depends on: D135952 Differential Revision: https://reviews.llvm.org/D135455 -
Sander de Smalen authored
When the SME attributes tell that a function is or may be executed in Streaming SVE mode, we currently need to be conservative and disable _any_ vectorization (fixed or scalable) because the code-generator does not yet support generating streaming-compatible code. Scalable auto-vec will be gradually enabled in the future when we have confidence that the loop-vectorizer won't use any SVE or NEON instructions that are illegal in Streaming SVE mode. Reviewed By: paulwalker-arm Differential Revision: https://reviews.llvm.org/D135950
-
Lorenzo Chelini authored
The assert is misplaced as the result type is allowed to be null. A few lines below the result type is inferred if it is passed a nullptr. Besides, this behavior is described in the documentation of the builder. Reviewed By: ftynse Differential Revision: https://reviews.llvm.org/D136262
-
- Oct 19, 2022
-
-
Chris Bieneman authored
Should have done this from the start. Since all the injected AST types are in the hlsl namespace we should also put the header-defined types and functions in there too. This updates the basic_types test to run once with the namespaced types and once without, and adds using declarations or namespaces calls in other tests. Reviewed By: python3kgae Differential Revision: https://reviews.llvm.org/D135973
-
Phoebe Wang authored
This is an alternative of D120395 and D120411. Previously we use `__bfloat16` as a typedef of `unsigned short`. The name may give user an impression it is a brand new type to represent BF16. So that they may use it in arithmetic operations and we don't have a good way to block it. To solve the problem, we introduced `__bf16` to X86 psABI and landed the support in Clang by D130964. Now we can solve the problem by switching intrinsics to the new type. Reviewed By: LuoYuanke, RKSimon Differential Revision: https://reviews.llvm.org/D132329
-
Jay Foad authored
This just commons up and simplifies some logic that was repeated in SIInsertWaitcnts::updateEventWaitcntAfter. NFCI. Differential Revision: https://reviews.llvm.org/D136253
-
Alexey Bataev authored
-
Malhar Jajoo authored
This patch is a partial fix for [[ https://github.com/llvm/llvm-project/issues/56349 | issue ]], due to functions affected by D117473. Implementation details: The patch essentially creates a new macro if the architecture is either intel32 or intel64, since the generate-def.pl cannot process boolean algebra on macros. Reviewed By: jlpeyton Differential Revision: https://reviews.llvm.org/D135795
-