- Jun 16, 2020
-
-
Louis Dionne authored
-
Louis Dionne authored
-
Erich Keane authored
-
Julian Lettner authored
The extended test fails on bots that still run lit tests with Python 2. https://reviews.llvm.org/D81316
-
Stanislav Mekhanoshin authored
Differential Revision: https://reviews.llvm.org/D81886
-
Jing Pu authored
Summary: FileLineColLoc allows the column and line to be zero to represent unknown column and/or unknown line/column information. However, SourceMgr::FindLocForLineAndColumn treats line 0 and col 0 valid and pointing to the first line and col, respectively. To adapt this mismatch in semantics, we explicitly check line/col being zeros in SourceMgrDiagnosticHandler::convertLocToSMLoc Differential Revision: https://reviews.llvm.org/D80258
-
Akira Hatanaka authored
parameters of non-trivial C struct special functions This removes the need to pass std::array of Addresses to getFunction, which were overwritten in the function.
-
River Riddle authored
This allows for passing a lambda to addDynamicallyLegalDialect without needing to explicit wrap with Optional<DynamicLegalityCallbackFn>. Differential Revision: https://reviews.llvm.org/D81680
-
River Riddle authored
It is quite common for the same type to be converted many types throughout the conversion process, and there isn't any good reason why we aren't caching that result. Especially given that we currently use identity conversion to signify legality. This revision also adds a few additional helpers to TypeConverter. Differential Revision: https://reviews.llvm.org/D81679
-
Mehdi Amini authored
-
Yaxun (Sam) Liu authored
Currently HIP toolchain calls clang to emit bitcode then calls opt/llc for device compilation for the default -fno-gpu-rdc case, which is unnecessary since clang is able to compile a single source file to ISA. This patch fixes the HIP action builder and toolchain so that the default -fno-gpu-rdc can be done like a canonical toolchain, i.e. one clang -cc1 invocation to compile source code to ISA. This can avoid unnecessary processes to speed up the compilation, and avoid redundant LLVM passes which are performed in clang -cc1 and opt. Differential Revision: https://reviews.llvm.org/D81627
-
Jonas Devlieghere authored
-
Nicolas Vasilache authored
This revision replaces MatmulOp, now that DRR rules have been dropped. This revision also fixes minor parsing bugs and a plugs a few holes to get e2e paths working (e.g. library call emission). During the replacement the i32 version had to be dropped because only the EDSC operators +, *, etc support type inference. Deciding on a type-polymorphic behavior, and implementing it, is left for future work. Differential Revision: https://reviews.llvm.org/D79762
-
Hanhan Wang authored
Differential Revision: https://reviews.llvm.org/D81858
-
Amara Emerson authored
-
Louis Dionne authored
The commit was reverted in 43c4afb5 because it broke the Windows to Linux cross-compilation build bots. The issue turned out to be that the bots were setting the LIBCXX_EXECUTOR incorrectly. This has been fixed now and verified with the bot owners. Note that this is only a partial re-application of the commit, since non-problematic parts of the commits have already been re-applied earlier.
-
Mircea Trofin authored
The initial intent was to organize ML stuff in its own directory, but it turns out that conflicts with llvm component layering policies: it is not a component, because subsequent changes want to rely on other analyses, which would create a cycle; and we don't have a reliable, cross-platform mechanism to compile files in a subdirectory, and fit in the existing LLVM build structure. This change moves the files into Analysis, and subsequent changes will leverage conditional compilation for those that have optional dependencies.
-
aartbik authored
Summary: Previous submit of new tests accidentally made this ON. The tests should be opt-in. To build with MLIR integration tests enabled, pass the following cmake .... \ -DMLIR_INCLUDE_INTEGRATION_TESTS=ON \ .... Reviewers: mehdi_amini Subscribers: mgorny, mehdi_amini, rriddle, jpienaar, shauheen, antiagainst, nicolasvasilache, arpith-jacob, mgester, lucyrfox, liufengdb, stephenneuendorffer, Joonsoo, grosul1, frgossen, Kayjukh, jurahul, msifontes Tags: #mlir Differential Revision: https://reviews.llvm.org/D81878
-
Nick Desaulniers authored
Summary: L is meant to support the second word used by 32b calling conventions for 64b arguments. This is required for build 32b PowerPC Linux kernels after upstream commit 334710b1496a ("powerpc/uaccess: Implement unsafe_put_user() using 'asm goto'") Thanks for the report from @nathanchance, and reference to GCC's implementation from @segher. Fixes: pr/46186 Fixes: https://github.com/ClangBuiltLinux/linux/issues/1044 Reviewers: echristo, hfinkel, MaskRay Reviewed By: MaskRay Subscribers: MaskRay, wuzish, nemanjai, hiraditya, kbarton, steven.zhang, llvm-commits, segher, nathanchance, srhines Tags: #llvm Differential Revision: https://reviews.llvm.org/D81767 -
Davide Italiano authored
The promotion machinery in CGP moves instructions retaining debug locations. When the transformation is local, this is mostly correct, but when instructions are moved cross-BBs, this is not always true and causes jumpiness in line tables. This is the first of a series of commits. sext(s) and zext(s) need to be treated similarly. Differential Revision: https://reviews.llvm.org/D81879
-
Jan Korous authored
-
Florian Hahn authored
As suggested in D81472, the load/store intrinsics' pointer arguments can be marked as nocapture and all matrix intrinsics as nosync. This also re-flows the intrinsic definitions, to make them a little more concise.
-
LLVM GN Syncbot authored
-
Sergej Jaskiewicz authored
The issue is that Windows doesn't support shebangs. This caused issues with libc++ tests.
-
Jan Korous authored
Differential Revision: https://reviews.llvm.org/D77179
-
Christopher Tetreault authored
Summary: Bail out in cstfp_pred_ty before calling getNumElements and walking the vector if the vector is scalable. Reviewers: efriedma, lebedev.ri, fhahn, sdesmalen, fpetrogalli Reviewed By: sdesmalen Subscribers: tschuett, rkruppe, psnobl, llvm-commits Tags: #llvm Differential Revision: https://reviews.llvm.org/D81617
-
Mehdi Amini authored
This is intended to avoid programming mistake where a temporary OpOperand is created, for example: for (OpOperand user : result.getUsers()) { It can be confusing for the user, in particular since in MLIR most classes are intended to be copied around by value while they have reference semantics. Differential Revision: https://reviews.llvm.org/D81815 -
Jessica Paquette authored
Apparently an x86 bot doesn't like the disabled rule in this test. http://lab.llvm.org:8011/builders/fuchsia-x86_64-linux/builds/6569 Remove disabled rule and update the test to try and pacify the bot.
-
Artem Belevich authored
It's useful for using clang from tools that may need need to provide SDK files from non-standard locations. Clang CLI only provides a way to specify VFS for include files, so there's no good way to test this yet. Differential Revision: https://reviews.llvm.org/D81771
-
Greg McGary authored
This is a complete Options.td compiled from ld(1) dated 2018-03-07 and cross checked with ld64 source code version 512.4 dated 2018-03-18. This is the first in a series of diffs for argument handling. Follow-ups will include switch cases for all the new instances of `OPT_foo`, and parsing/validation of arguments attached to options, e.g., more code akin to `OPT_platform_version` and associated `parsePlatformVersion()`. Reviewed By: smeenai Differential Revision: https://reviews.llvm.org/D80582
-
Nikita Popov authored
This wraps the uint8_t[12] type used in two places, because I plan to introduce a third use of the same pattern.
-
Shoaib Meenai authored
The case the calculation works for is when r_length = 2.
-
Arnold Schwaighofer authored
Summary: If a record has a mix of relative pointers and other fields they wouldn't necessarily be the same. Fallout from D77592. rdar://64309883 Subscribers: cfe-commits Tags: #clang Differential Revision: https://reviews.llvm.org/D81857
-
Jessica Paquette authored
Add selection support for ext via a new opcode, G_EXT and a post-legalizer combine which matches it. Add an `applyEXT` function, because the AArch64ext patterns require a register for the immediate. So, we have to create a G_CONSTANT to get these without writing new patterns or modifying the existing ones. Tests are the same as arm64-ext.ll. Also prevent ext from firing on the zip test. It has higher priority, so we don't want it potentially getting in the way of mask tests. Also fix up the shuffle-splat test, because ext is now selected there. The test was incorrectly regbank selected before, which could cause a verifier failure when you emit copies. Differential Revision: https://reviews.llvm.org/D81436
-
Mircea Trofin authored
This reverts commit 695c7d63. Breaks windows (e.g. http://lab.llvm.org:8011/builders/clang-x64-windows-msvc/builds/16497) Likely to cause problems with XCode.
-
Davide Italiano authored
Pointed out by dblaikie.
-
Mircea Trofin authored
Summary: Currently, add_llvm_library would create an OBJECT library alongside of a STATIC / SHARED library, but losing the link interface (its elements would become dependencies instead). To support scenarios where linking an object library also brings in its usage requirements, this patch adds support for 'stand-alone' OBJECT libraries - i.e. without an accompanying SHARED/STATIC library, and maintaining the link interface defined by the user. The support is via a new option, OBJECT_ONLY, to avoid breaking changes - since just specifying "OBJECT" would currently imply also STATIC or SHARED, depending on BUILD_SHARED_LIBS. This is useful for cases where, for example, we want to build a part of a component separately. Using a STATIC target would incur the risk that symbols not referenced in the consumer would be dropped (which may be undesirable). The current application is the ML part of Analysis. It should be part of the Analysis component, so it may reference other analyses; and (in upcoming changes) it has dependencies on optional libraries. Reviewers: karies, davidxl Subscribers: mgorny, hiraditya, llvm-commits Tags: #llvm Differential Revision: https://reviews.llvm.org/D81447
-
Matt Arsenault authored
This is a somewhat artifical limit, so avoid repeating it many places in case it changes.
-
Louis Dionne authored
This is useful for checking runtime properties of the target system. This is a partial re-application of 3ea9450b. This part was tested to work on a Windows host with a SSH executor.
-
Matt Arsenault authored
-