summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
* Reland '[InstructionCombining] Fixed null check after dereferencing warning. ↵Dávid Bolvanský2019-11-031-2/+5
| | | | NFCI.'
* Revert "[InstructionCombining] Fixed null check after dereferencing warning. ↵Dávid Bolvanský2019-11-031-1/+0
| | | | | | NFCI." This reverts commit 8308187fd9bfa08ffad0a636d4dd1d25e7de5a76. This exposed a bug.
* [SCEV] Fixed 'Uninitialized variable 'ContainsAddRec' used.' warning. NFCI.Dávid Bolvanský2019-11-031-1/+1
|
* [MemorySSA] Fixed null check after dereferencing warning. NFCI.Dávid Bolvanský2019-11-031-0/+1
|
* Revert "[InstructionCompares] Fixed null check after dereferencing warning. ↵Dávid Bolvanský2019-11-031-1/+0
| | | | | | NFCI." This reverts commit b8685cf3042f6a2e129061922bd6b18e3c42258e.
* [InstructionCompares] Fixed null check after dereferencing warning. NFCI.Dávid Bolvanský2019-11-031-0/+1
|
* [InstructionCombining] Fixed null check after dereferencing warning. NFCI.Dávid Bolvanský2019-11-031-0/+1
|
* [CHR] Fixed null check after dereferencing warning. NFCI.Dávid Bolvanský2019-11-031-1/+1
|
* [LoopUnrollRuntime] Fixed null check after dereferencing warning. NFCI.Dávid Bolvanský2019-11-031-2/+2
|
* [LoopUnrollAndJam] Fixed null check after dereferencing warning. NFCI.Dávid Bolvanský2019-11-031-1/+3
|
* [BitcodeReader] Fixed use after move warnings. NFCI.Dávid Bolvanský2019-11-031-10/+0
|
* [BitcodeReader] Fixed null check after dereferencing warning. NFCI.Dávid Bolvanský2019-11-031-1/+1
|
* [BitcodeReader] Fixed null pointer dereferencing warning. NFCI.Dávid Bolvanský2019-11-031-0/+1
|
* [SelectionDAG] Fixed null check after dereferencing warning. NFCI.Dávid Bolvanský2019-11-031-1/+1
|
* [opaque pointer types] Add element type argument to IRBuilder ↵Craig Topper2019-11-034-13/+11
| | | | | | | | | | | | | | | | | | | | CreatePreserveStructAccessIndex and CreatePreserveArrayAccessIndex Summary: These were the only remaining users of the GetElementPtrInst::getGEPReturnType method that gets the element type from the pointer type. Remove that method since its now dead. Reviewers: jyknight, t.p.northover, arsenm Reviewed By: arsenm Subscribers: wdng, arsenm, arphaman, cfe-commits, llvm-commits Tags: #clang, #llvm Differential Revision: https://reviews.llvm.org/D69756
* gn build: (manually) merge 3a399c09 / add76dd3cNico Weber2019-11-031-2/+0
|
* [X86][SSE] combineX86ShufflesRecursively - don't bother merging shuffles ↵Simon Pilgrim2019-11-031-92/+105
| | | | | | with empty roots. NFCI. This doesn't affect actual codegen, but is a minor refactor toward fixing PR43024 where we need to avoid excess changes (folding zeroables etc.) to the shuffle mask at Depth == 0.
* [X86] Convert PICStyles::Style to scoped enum class. NFCI.Simon Pilgrim2019-11-032-11/+11
| | | | Fixes MSVC static analyzer warnings about enum safety, this enum performs no integer math so it'd be better to fix its scope.
* [DebugInfo] Fix for DW_OP_LLVM_fragment in DIExpression::isImplicit()Bjorn Pettersson2019-11-032-9/+102
| | | | | | | | | | | DIExpression::isImplicit() did not handle DW_OP_LLVM_fragment correctly. It was scanning the elements in the expression by iterating from the end. But we do not know the position of ops unless we iterate from the beginning of the expression, since DW_OP:s and their operands are stored flat in the expression list. The old code also assumed that a DW_OP_LLVM_fragment only occupied one element in the expression list, but it actually occupies three elements.
* [BPF] fix a bug in __builtin_preserve_field_info() with FIELD_BYTE_SIZEYonghong Song2019-11-032-1/+87
| | | | | | | | | | | | | | | | | | | | | | | | During deriving proper bitfield access FIELD_BYTE_SIZE, function Member->getStorageOffsetInBits() is used to get llvm IR type storage offset in bits so that the byte size can permit aligned loads/stores with previously derived FIELD_BYTE_OFFSET. The function should only be used with bitfield members and it will assert if ASSERT is turned on during cmake build. Constant *getStorageOffsetInBits() const { assert(getTag() == dwarf::DW_TAG_member && isBitField()); if (auto *C = cast_or_null<ConstantAsMetadata>(getExtraData())) return C->getValue(); return nullptr; } This patch fixed the issue by using Member->isBitField() directly and a test case is added to cover this missing case. This issue is discovered when running Andrii's linux kernel CO-RE tests. Differential Revision: https://reviews.llvm.org/D69761
* [compiler-rt] Harmonize __sanitizer_addrinfo with the NetBSD headersKamil Rytarowski2019-11-031-0/+6
| | | | Add missing pad for sparc, alpha and a variation of i386.
* [compiler-rt] Sync NetBSD syscall hooks with 9.99.17Kamil Rytarowski2019-11-033-57/+214
| | | | | Document the minimal version supported as 9.0 and add compat code for renamed syscalls after 9.0.
* Drop spurious self-include [NFC]Aaron Puchert2019-11-031-1/+0
| | | | This was introduced in D61357, probably by accident.
* ModuleMap::findHeader - fix null dereference warning. NFCI.Simon Pilgrim2019-11-031-1/+1
| | | | We were checking M for a null value after we'd already dereferenced it multiple times.
* SymbolRecord - fix more uninitialized variable warnings. NFCI.Simon Pilgrim2019-11-031-14/+14
|
* Fix uninitialized variable warnings. NFCI.Simon Pilgrim2019-11-032-3/+7
|
* Fix line_iterator uninitialized variable warnings. NFCI.Simon Pilgrim2019-11-031-5/+5
| | | | Allows us to auto define the default constructor as well.
* Ensure VPlanPrinter::Depth is initialized to fix static analyzer warning. NFCI.Simon Pilgrim2019-11-031-1/+1
|
* Fix uninitialized variable warning. NFCI.Simon Pilgrim2019-11-031-1/+1
|
* [CodeGen] [ExpandReduction] Fix the bug for ExpandReduction() when vector ↵shkzhang2019-11-022-0/+19
| | | | | | | | | | | | | | | | | | | | size isn't power of 2 Summary: For below test case, we will get assert error except for AArch64 and ARM: declare i8 @llvm.experimental.vector.reduce.and.i8.v3i8(<3 x i8> %a) define i8 @test_v3i8(<3 x i8> %a) nounwind { %b = call i8 @llvm.experimental.vector.reduce.and.i8.v3i8(<3 x i8> %a) ret i8 %b } In the function getShuffleReduction (), we can see it needs the vector size must be power of 2. This patch is fix below error when the number of element is not power of 2 for those llvm.experimental.vector.reduce.* function. Reviewed By: jsji Differential Revision: https://reviews.llvm.org/D68625
* [CodeGenCXX][test] Use -fno-experimental-new-pass-manager for ↵Fangrui Song2019-11-021-1/+1
| | | | | | CodeGenCXX/union-tbaa2.cpp after D68593/llvmorg-10-init-8907-gcecc0d27ad5 It fails with -DENABLE_EXPERIMENTAL_NEW_PASS_MANAGER=0 builds. Temporarily use -fno-experimental-new-pass-manager while we are investigating the root cause.
* Stop static analyzer warnings about using bitwise operators on booleans. NFCI.Simon Pilgrim2019-11-021-6/+6
| | | | Call each of the rebase_if() calls separately.
* TargetParserTest - testARMArch - stop bitwise operators on boolean warnings. ↵Simon Pilgrim2019-11-021-4/+5
| | | | | | NFCI. Ensure the test still runs all target checks but stop the static analyzer warnings.
* CodeGen/DIE.h: prefer the default member initializer to the member ↵Fangrui Song2019-11-021-2/+2
| | | | initializers in the constructor. NFC
* [mips] Remove trailing spaces. NFCSimon Atanasyan2019-11-031-4/+4
|
* [mips] Split long lines in the code. NFCSimon Atanasyan2019-11-0328-143/+216
|
* A15SDOptimizer::getPrefSPRLane - fix null dereference warning. NFCISimon Pilgrim2019-11-021-2/+1
|
* isConditionalBranch/isUnconditionalBranch - use boolean operators. NFCI.Simon Pilgrim2019-11-022-4/+4
| | | | Stop static analyzer warnings about using bitwise operators on booleans.
* [LV] Generalize conditions for sinking instrs for first order recurrences.Florian Hahn2019-11-022-14/+271
| | | | | | | | | | | | | | | | | | | | | | | | | If the recurrence PHI node has a single user, we can sink any instruction without side effects, given that all users are dominated by the instruction computing the incoming value of the next iteration ('Previous'). We can sink instructions that may cause traps, because that only causes the trap to occur later, but not on any new paths. With the relaxed check, we also have to make sure that we do not have a direct cycle (meaning PHI user == 'Previous), which indicates a reduction relation, which potentially gets missed by ReductionDescriptor. As follow-ups, we can also sink stores, iff they do not alias with other instructions we move them across and we could also support sinking chains of instructions and multiple users of the PHI. Fixes PR43398. Reviewers: hsaito, dcaballe, Ayal, rengolin Reviewed By: Ayal Differential Revision: https://reviews.llvm.org/D69228
* isImmPCRel/isImmSigned - both functions should return bool not unsigned. NFCI.Simon Pilgrim2019-11-021-2/+2
|
* MatchTableRecord::emit - fix boolean operator precedence warnings from PVS ↵Simon Pilgrim2019-11-021-2/+2
| | | | | | Studio. NFCI. Make it clear that (Flags & MTRF_????) should resolve to a boolean.
* [Attributor][NFCI] Do not track unnecessary dependencesJohannes Doerfert2019-11-021-3/+8
| | | | | If we do not look at assumed information there is no need to track dependences.
* [Attributor][NFCI] Distinguish optional and required dependencesJohannes Doerfert2019-11-029-30/+93
| | | | | | | | | | | | | | | | | | Dependences between two abstract attributes SRC and TRG come naturally in two flavors: Either (1) "some" information of SRC is *required* for TRG to derive information, or (2) SRC is just an *optional* way for TRG to derive information. While it is not strictly necessary to distinguish these types explicitly, it can help us to converge faster, in terms of iterations, and also cut down the number of `AbstractAttribute::update` calls. As far as I can tell, we only use optional dependences for liveness so far but that might change in the future. With this change the Attributor can be informed about the "dependence class" and it will perform appropriate actions when an Attribute is set to an invalid state, thus one that cannot be used by others to derive information from.
* Fix -Wreorder warnings. NFCI.Simon Pilgrim2019-11-021-1/+1
|
* ARMAttributeParser - fix shadow variable name warnings from decodeULEB128 ↵Simon Pilgrim2019-11-021-9/+9
| | | | | | calls. NFCI. Consistently rename the Length attribute to DecodeLength in decodeULEB128 calls.
* GSYMTest::TestLineTable - check all LT1+LT2 comparisons.Simon Pilgrim2019-11-021-0/+2
| | | | PVS Studio was warning about "LT2 < LT2" but really we should be testing all permutations of LT1 and LT2.
* [LegacyPassManager] Fixed "null check after derefencing" warningDávid Bolvanský2019-11-021-2/+1
| | | | The 'RequiredPass' pointer was utilized before it was verified against nullptr. Check lines: 1626, 1629.
* build: avoid custom handling for C++ standardSaleem Abdulrasol2019-11-021-14/+11
| | | | | | Use the builtin CMake support for specifying the proper flags for the targets to build at a certain C++ standard. This avoids unnecessary checks in CMake, speeding up the configure phase as well as simplifies the logic overall.
* X86_MC::createX86MCSubtargetInfo - X86_MC::ParseX86Triple never returns an ↵Simon Pilgrim2019-11-021-6/+3
| | | | | | empty string. NFCI. PVS Studio was complaining that the expression '!ArchFS.empty()' is always true.
* X86Operand::print - fix SymName shadow variable warning. NFCI.Simon Pilgrim2019-11-021-2/+2
|
OpenPOWER on IntegriCloud