- Apr 12, 2020
-
-
Simon Pilgrim authored
-
Simon Pilgrim authored
Remove Pass.h include - we don't use anything direct from the header and its implicitly included by MachineFunctionPass.h. Remove unused forward declarations.
-
Simon Pilgrim authored
We need to include memory.h as it was being implicitly included. Also remove unused llvm::TargetOptions forward declaration.
-
Florian Hahn authored
struct should be used when all members are public: https://llvm.org/docs/CodingStandards.html#use-of-class-and-struct-keywords Reviewers: gilr, rengolin, Ayal, hsaito Reviewed By: Ayal Differential Revision: https://reviews.llvm.org/D77865
-
Simon Pilgrim authored
-
Ehud Katz authored
Define MLIR_MAIN_INCLUDE_DIR, as it was not set anywhere. Set MLIR_MAIN_SRC_DIR to the actual "source directory", and not the "include directory" (as currently set). Differential Revision: https://reviews.llvm.org/D77943
-
River Riddle authored
Summary: This revision makes the registration of command line options for these two files manual with `registerMLIRContextCLOptions` and `registerAsmPrinterCLOptions` methods. This removes the last remaining static constructors within lib/. Differential Revision: https://reviews.llvm.org/D77960
-
River Riddle authored
Summary: std::function has a notoriously large amount of malloc traffic, whereas function_ref is a cheaper and more efficient alternative. Differential Revision: https://reviews.llvm.org/D77959
-
Chris Lattner authored
the indentation of the StringSet.h file and its file comment header, and significantly reduce redundant #includes that are already pulled in transitively. NFC. This is in preparation for a more interesting patch I'll post to phab.
-
Chris Lattner authored
StringMap<char>, saving a byte due to the StringMapEntry specialization for a value of NoneType. NFC.
-
Stephen Neuendorffer authored
A few libraries which are also Dialect libraries where independently in the link line for mlir-opt. Remove them. Differential Revision: https://reviews.llvm.org/D77927
-
Stephen Neuendorffer authored
Differential Revision: https://reviews.llvm.org/D77926
-
Chris Lattner authored
with DenseMapInfo<mlir::Identifier>::getHashValue. NFC. Adjust a few variable names to follow MLIR convention while here NFC.
-
Chris Lattner authored
Summary: Identifier doesn't maintain a length, so every time strref() is called, it does a strlen. In the case of comparisons, this isn't necessary: there is no need to scan a string to get its length, then rescan it to do the comparison. Just done one comparison. This also moves some assertions in Identifier::get as another microoptimization for 'assertions enabled' modes. Reviewers: rriddle! Subscribers: mehdi_amini, rriddle, jpienaar, burmako, shauheen, antiagainst, nicolasvasilache, arpith-jacob, mgester, lucyrfox, liufengdb, Joonsoo, grosul1, frgossen, llvm-commits Tags: #llvm Differential Revision: https://reviews.llvm.org/D77958
-
Craig Topper authored
Differential Revision: https://reviews.llvm.org/D77933
-
Craig Topper authored
A lot of vectorized code doesn't use masks so we shouldn't penalize them by not doing shuffle combining on avx512 targets. I've added support for VALIGNQ/VALIGND and 512-bit SHUF128 to prevent some regressions. I also prevented recombining 256-bit SHUF128 to PERM2X128. We may not need to add 256-bit SHUF128 support, but I don't think I found any cases requiring that in my testing. Differential Revision: https://reviews.llvm.org/D77928
-
Stefanos Baziotis authored
-
Sam Clegg authored
The actual bug was fixed in https://reviews.llvm.org/D74589 Differential Revision: https://reviews.llvm.org/D77084
-
Matt Arsenault authored
These need to be promoted and scalarized for the SALU.
-
David Blaikie authored
Broke an LLDB build bot & I can't seem to build LLDB locally to fix forward... http://lab.llvm.org:8011/builders/lldb-x64-windows-ninja/builds/15567/steps/test/logs/stdio This reverts commit 416fa772.
-
Rahman Lavaee authored
This reverts commit ab31797e.
-
Rahman Lavaee authored
-
Matt Arsenault authored
The current selector depends on 16-bit shifts using 16-bit shift amount types, but really it should accept either for all types.
-
Uday Bondhugula authored
Add doc cross links between std.alloca and AutomaticAllocationScope. Subscribers: mehdi_amini, rriddle, jpienaar, burmako, shauheen, antiagainst, nicolasvasilache, arpith-jacob, mgester, lucyrfox, liufengdb, Joonsoo, grosul1, frgossen, llvm-commits Tags: #llvm Differential Revision: https://reviews.llvm.org/D77956
-
Craig Topper authored
-Drop llvm:: on MVT::i32 -Use getValueType instead of getSimpleValueType for an equality check just cause its shorter and doesn't matter. -Don't create a const SDValue & since its cheap to copy. -Remove explicit case from MVT enum to EVT. -Add message to assert.
-
Craig Topper authored
[X86] Move code that replaces ISD::VSELECT with X86ISD::BLENDV from X86DAGToDAGISel::Select to PreprocessISelDAG
-
Eli Friedman authored
This would have saved me some time this morning.
-
River Riddle authored
This prefix is already implied by the doc location and functions only as filler.
-
Simon Pilgrim authored
Replace SmallSet.h include with SmallVector.h - only SmallVector.h is used from inside SmallSet.h Remove llvm::StringRef forward declaration (we include StringRef.h) Remove unused llvm::GlobalVariable forward declaration.
-
Kadir Cetinkaya authored
-
Craig Topper authored
[CallSite removal][SelectionDAGBuilder] Use CallBase instead of ImmutableCallSite in visitPatchpoint. Differential Revision: https://reviews.llvm.org/D77932
-
Kadir Cetinkaya authored
Summary: We had tests in clangd (target_info.test) that got enabled only on systems that know about x86. But they were always disabled as clangd lit config never registered those targets. This patch adds those targets as `$TARGET$-registered-target` Reviewers: sammccall Subscribers: ilya-biryukov, MaskRay, jkorous, arphaman, usaxena95, cfe-commits Tags: #clang Differential Revision: https://reviews.llvm.org/D77944
-
Matt Arsenault authored
-
Matt Arsenault authored
-
Hongtao Yu authored
Summary: A count profile may affect tail duplication's heuristic causing a block to be duplicated in only a part of its predecessors. This is not allowed in the Machine Block Placement pass where an assert will go off. I'm removing the assert and making the optimization bail out when such case happens. Reviewers: wenlei, davidxl, Carrot Reviewed By: Carrot Subscribers: hiraditya, llvm-commits Tags: #llvm Differential Revision: https://reviews.llvm.org/D77748
-
Aaron Puchert authored
Summary: If we don't read the flag, we don't need to set it. Then we also don't need to filter it out to get the pointer. This should have a (small) positive impact on code size and performance. Reviewed By: lhames Differential Revision: https://reviews.llvm.org/D76259
-
River Riddle authored
Summary: This revision performs a few refactorings on the main docs folder. Namely it: * Adds a new Rationale/ folder to contain various rationale documents * Moves several "getting started" documents to the Tutorials/ folder * Cleans up the titles of various documents Differential Revision: https://reviews.llvm.org/D77934
-
Casey Carter authored
-
Fangrui Song authored
-
Fangrui Song authored
-