Revision
359988
by ctopper:
[LLParser] Remove unused variable after r359987. NFC |
Change Type | Path in Repository | Path in Workspace |
---|
 | /llvm/trunk/lib/AsmParser/LLParser.cpp | trunk/lib/AsmParser/LLParser.cpp |
Revision
359987
by ctopper:
[LLParser] Remove unnecessary error check making sure NUW/NSW flags aren't set on a non-integer operation. Summary: This check appears to be a leftover from when add/sub/mul could be either integer or fp. The NSW/NUW flags are only set for add/sub/mul/shl earlier. And we check that those operations only have integer types just below this. So it seems unnecessary to explicitly error for NUW/NSW being used on a add/sub/mul that have the wrong type that would later error for that. Reviewers: spatel, dblaikie, jyknight, arsenm Reviewed By: spatel Subscribers: wdng, llvm-commits, hiraditya Tags: #llvm Differential Revision: https://reviews.llvm.org/D61562 |
Change Type | Path in Repository | Path in Workspace |
---|
 | /llvm/trunk/lib/AsmParser/LLParser.cpp | trunk/lib/AsmParser/LLParser.cpp |
Revision
359986
by ctopper:
[LLParser] Simplify type checking in ParseArithmetic and ParseUnaryOp. Summary: These methods previously took a 0, 1, or 2 to indicate what types were allowed, but the 0 encoding which meant both fp and integer types has been unused for years. Its leftover from when add/sub/mul used to be shared between int and fp Simplify it by changing it to just a bool to distinquish int and fp. Reviewers: spatel, dblaikie, jyknight, arsenm Reviewed By: spatel Subscribers: wdng, hiraditya, llvm-commits Tags: #llvm Differential Revision: https://reviews.llvm.org/D61561 |
Change Type | Path in Repository | Path in Workspace |
---|
 | /llvm/trunk/lib/AsmParser/LLParser.cpp | trunk/lib/AsmParser/LLParser.cpp |
 | /llvm/trunk/lib/AsmParser/LLParser.h | trunk/lib/AsmParser/LLParser.h |
Revision
359985
by ctopper:
[Constants] Simplify type checking switch in ConstantExpr::get. Summary: Remove duplicate checks that both operands have the same type. This is checked before the switch. Use 'integer' or 'floating-point' instead of 'arithmetic' type. I think this might be a leftover to the days when floating point and integer operations shared the same opcodes. Reviewers: spatel, RKSimon, dblaikie Reviewed By: RKSimon Subscribers: hiraditya, llvm-commits Tags: #llvm Differential Revision: https://reviews.llvm.org/D61558 |
Change Type | Path in Repository | Path in Workspace |
---|
 | /llvm/trunk/lib/IR/Constants.cpp | trunk/lib/IR/Constants.cpp |
Revision
359983
by adibiagio:
[MCA] Notify event listeners when instructions transition to the Pending state. NFCI |
Change Type | Path in Repository | Path in Workspace |
---|
 | /llvm/trunk/include/llvm/MCA/HWEventListener.h | trunk/include/llvm/MCA/HWEventListener.h |
 | /llvm/trunk/include/llvm/MCA/HardwareUnits/Scheduler.h | trunk/include/llvm/MCA/HardwareUnits/Scheduler.h |
 | /llvm/trunk/include/llvm/MCA/Stages/ExecuteStage.h | trunk/include/llvm/MCA/Stages/ExecuteStage.h |
 | /llvm/trunk/lib/MCA/HardwareUnits/Scheduler.cpp | trunk/lib/MCA/HardwareUnits/Scheduler.cpp |
 | /llvm/trunk/lib/MCA/Stages/ExecuteStage.cpp | trunk/lib/MCA/Stages/ExecuteStage.cpp |
|
Change Type | Path in Repository | Path in Workspace |
---|
 | /llvm/trunk/include/llvm/Analysis/ConstantFolding.h | trunk/include/llvm/Analysis/ConstantFolding.h |
 | /llvm/trunk/lib/Analysis/ConstantFolding.cpp | trunk/lib/Analysis/ConstantFolding.cpp |
 | /llvm/trunk/lib/IR/ConstantFold.cpp | trunk/lib/IR/ConstantFold.cpp |
 | /llvm/trunk/lib/IR/ConstantFold.h | trunk/lib/IR/ConstantFold.h |
 | /llvm/trunk/lib/IR/Constants.cpp | trunk/lib/IR/Constants.cpp |
 | /llvm/trunk/test/Analysis/ConstantFolding/fneg.ll | trunk/test/Analysis/ConstantFolding/fneg.ll |
 | /llvm/trunk/test/Transforms/InstCombine/fneg.ll | trunk/test/Transforms/InstCombine/fneg.ll |
 | /llvm/trunk/test/Transforms/InstCombine/fsub.ll | trunk/test/Transforms/InstCombine/fsub.ll |
 | /llvm/trunk/test/Transforms/InstCombine/inselt-binop.ll | trunk/test/Transforms/InstCombine/inselt-binop.ll |
 | /llvm/trunk/test/Transforms/Reassociate/crash2.ll | trunk/test/Transforms/Reassociate/crash2.ll |
 | /llvm/trunk/unittests/IR/ConstantsTest.cpp | trunk/unittests/IR/ConstantsTest.cpp |
Revision
359981
by rksimon:
[X86] Make X86RegisterInfo(const Triple &TT) constructor explicit. Fixes cppcheck warning. |
Change Type | Path in Repository | Path in Workspace |
---|
 | /llvm/trunk/lib/Target/X86/X86RegisterInfo.h | trunk/lib/Target/X86/X86RegisterInfo.h |
Revision
359976
by rksimon:
[X86] Fix some cppcheck "Local variable name shadows outer variable" warnings. NFCI. |
Change Type | Path in Repository | Path in Workspace |
---|
 | /llvm/trunk/lib/Target/X86/X86ISelLowering.cpp | trunk/lib/Target/X86/X86ISelLowering.cpp |
Revision
359975
by rksimon:
[SLPVectorizer] Make getSpillCost() const. NFCI. Ideally getTreeCost() should be const as well but non-const Type creation would need to be addressed first. |
Change Type | Path in Repository | Path in Workspace |
---|
 | /llvm/trunk/lib/Transforms/Vectorize/SLPVectorizer.cpp | trunk/lib/Transforms/Vectorize/SLPVectorizer.cpp |
Revision
359974
by rksimon:
[SelectionDAG] Use any_of/all_of where possible. NFCI. |
Change Type | Path in Repository | Path in Workspace |
---|
 | /llvm/trunk/lib/CodeGen/SelectionDAG/SelectionDAG.cpp | trunk/lib/CodeGen/SelectionDAG/SelectionDAG.cpp |
Revision
359973
by rksimon:
Move Value *RHSCIOp def into the scope where its actually used. NFCI. |
Change Type | Path in Repository | Path in Workspace |
---|
 | /llvm/trunk/lib/Transforms/InstCombine/InstCombineCompares.cpp | trunk/lib/Transforms/InstCombine/InstCombineCompares.cpp |
Revision
359970
by mcinally:
Add InstCombine tests for FNeg instruction. |
Change Type | Path in Repository | Path in Workspace |
---|
 | /llvm/trunk/test/Transforms/InstCombine/fneg.ll | trunk/test/Transforms/InstCombine/fneg.ll |
Revision
359969
by spatel:
[CodeGenPrepare] limit overflow intrinsic matching to a single basic block (2nd try) This is a subset of the original commit from rL359879 which was reverted because it could crash when using the 'RemovedInstructions' structure that enables delayed deletion of dead instructions. The motivating compile-time win does not require that change though. We should get most of that win from this change alone. Using/updating a dominator tree to match math overflow patterns may be very expensive in compile-time (because of the way CGP uses a DT), so just handle the single-block case. See post-commit thread for rL354298 for more details: http://lists.llvm.org/pipermail/llvm-commits/Week-of-Mon-20190422/646276.html Differential Revision: https://reviews.llvm.org/D61075 |
Change Type | Path in Repository | Path in Workspace |
---|
 | /llvm/trunk/lib/CodeGen/CodeGenPrepare.cpp | trunk/lib/CodeGen/CodeGenPrepare.cpp |
 | /llvm/trunk/test/CodeGen/X86/cgp-usubo.ll | trunk/test/CodeGen/X86/cgp-usubo.ll |
 | /llvm/trunk/test/Transforms/CodeGenPrepare/X86/optimizeSelect-DT.ll | trunk/test/Transforms/CodeGenPrepare/X86/optimizeSelect-DT.ll |
 | /llvm/trunk/test/Transforms/CodeGenPrepare/X86/overflow-intrinsics.ll | trunk/test/Transforms/CodeGenPrepare/X86/overflow-intrinsics.ll |
Revision
359965
by maskray:
[llvm-nm] Convert weak.test to use yaml2obj and fix unntested 'v' This restores part of the good change reverted by r359830. |
Change Type | Path in Repository | Path in Workspace |
---|
 | /llvm/trunk/test/tools/llvm-nm/X86/Inputs/weak.obj.elf-x86_64 | trunk/test/tools/llvm-nm/X86/Inputs/weak.obj.elf-x86_64 |
 | /llvm/trunk/test/tools/llvm-nm/X86/weak.test | trunk/test/tools/llvm-nm/X86/weak.test |
|
Change Type | Path in Repository | Path in Workspace |
---|
 | /llvm/trunk/lib/Target/AMDGPU/GCNHazardRecognizer.cpp | trunk/lib/Target/AMDGPU/GCNHazardRecognizer.cpp |