- Sep 21, 2022
-
-
Chen Zheng authored
This solves https://github.com/llvm/llvm-project/issues/57664 Reviewed By: sameerds Differential Revision: https://reviews.llvm.org/D134019
-
Michał Górny authored
Fix teletype formatting in configuration file documentation to use double backticks rather than single backticks. Cover some more names with this formatting. Correct the name of config file for `clang-cl` invocation. Differential Revision: https://reviews.llvm.org/D134271
-
Jennifer Yu authored
This patch add codegen support for the has_device_addr clause. It use the same logic of is_device_ptr. But passing &var instead pointer to var to kernal. Differential Revision: https://reviews.llvm.org/D134268
-
Mehdi Amini authored
This reverts commit 5d4603a0. The Dialect/Tensor/fold-consecutive-insert-extract-slice.mlir test is failing when built with GCC
-
Kevin Gleason authored
Discovered an issue working in StableHLO when attempting to build with `-Wall -Werror`: https://github.com/openxla/stablehlo/pull/137 Currently, if `LLVM_USE_LLD` and `-DCMAKE_CXX_FLAGS="${CMAKE_CXX_FLAGS} -Wall -Werror"` are both specified for build, the build will error with: ``` -- Performing Test CXX_SUPPORTS_CUSTOM_LINKER - Failed CMake Error at /usr/local/google/home/gleasonk/Coding/llvm-build/lib/cmake/llvm/HandleLLVMOptions.cmake:309 (message): Host compiler does not support '-fuse-ld=lld' ... $ cat <build_dir>/CMakeFiles/CMakeError.log ... clang: error: argument unused during compilation: '-fuse-ld=lld' [-Werror,-Wunused-command-line-argument] ``` It looks like other repos have hit this same issue: - https://github.com/golang/go/issues/41527 (mentioned in comment) - https://github.com/iree-org/iree/pull/7450 This can be reproduced in llvm-project with the following build command: ``` # Compile command taken from https://mlir.llvm.org/getting_started/ # and modified for use case cmake -G Ninja ../llvm \ -DLLVM_ENABLE_PROJECTS=mlir \ -DLLVM_BUILD_EXAMPLES=ON \ -DLLVM_TARGETS_TO_BUILD="X86;NVPTX;AMDGPU" \ -DCMAKE_BUILD_TYPE=Release \ -DLLVM_ENABLE_ASSERTIONS=ON \ -DCMAKE_CXX_COMPILER=clang++ \ -DCMAKE_C_COMPILER=clang \ -DLLVM_ENABLE_LLD=ON \ -DCMAKE_CXX_FLAGS="${CMAKE_CXX_FLAGS} -Wall -Werror" ``` Reviewed By: mehdi_amini Differential Revision: https://reviews.llvm.org/D134206
-
Christopher Bate authored
This change allows the SCF LoopPipelining transform to handle ops with nested regions within the pipelined `scf.for` body. The op and nested regions are treated as a single unit from the transform's perspective. This change also makes explicit the requirement that only ops whose parent Block is the loop body Block are allowed to be scheduled by the caller. Reviewed By: ThomasRaoux, nicolasvasilache Differential Revision: https://reviews.llvm.org/D133965
-
Emmmer authored
-
Craig Topper authored
This extension does not appear to be on its way to ratification. Out of the unratified bitmanip extensions, this one had the largest impact on the compiler. Posting this patch to start a discussion about whether we should remove these extensions. We'll talk more at the RISC-V sync meeting this Thursday. Reviewed By: asb, reames Differential Revision: https://reviews.llvm.org/D133834
-
Fangrui Song authored
[Driver] Fix -f[no-]unwind-tables -Wunused-command-line-argument after 4388b56d
-
jacquesguan authored
[RISCV] Support peephole optimization to fold vmerge.vvm that has tail agnostic policy and unmasked intrinsics. This patch supports the tail agnostic part of D130442. Reviewed By: fakepaper56 Differential Revision: https://reviews.llvm.org/D132923
-
Ting Wang authored
Reviewed By: lkail Differential Revision: https://reviews.llvm.org/D133921
-
Volodymyr Sapsai authored
[modules] Fix error "malformed or corrupted AST file: 'SourceLocation remap refers to unknown module...'". When a framework can be found at a new location, all references to it in the module cache become outdated. When we try to load such outdated .pcm file, we shouldn't change any already loaded and processed modules. If `Module` has `ASTFile`, it means we've read its AST block already and it is too late to undo that. If `ASTFile` is `None`, there is no value in setting it to `None` again. So we don't reset `ASTFile` in `ModuleManager::removeModules` at all. rdar://97216258 Differential Revision: https://reviews.llvm.org/D134249
-
Kazu Hirata authored
This patch fixes: clang/lib/Driver/ToolChains/PS4CPU.cpp:159:14: error: unused variable 'IsPS5' [-Werror,-Wunused-variable]
-
Changpeng Fang authored
Summary: Under code object version 5, ockl_get_local_size returns the value computed by the expression: workgroup_id < hidden_block_count ? hidden_group_size : hidden_remainder For functions with the attribute uniform-work-group-size=true. we can evaluate workgroup_id < hidden_block_count as true, and thus hidden_group_size is returned for ockl_get_local_size. With uniform-workgroup-size=true, this work also set all remainders to zero, and if there is reqd_work_group_size, we also set work-group-size to the required value from the metadata. Reviewers: arsenm and bcahoon Differential Revision: https://reviews.llvm.org/D131276
-
Lei Zhang authored
Use a suffix to make clear what the contents inside each file are. Reviewed By: ThomasRaoux Differential Revision: https://reviews.llvm.org/D134202
-
Lei Zhang authored
This commit adds utility functions to perform general merging of OffsetSizeAndStrideOpInterface by supporting producer rank reducing and non-unit strides. With it we can extend MergeConsecutiveExtractSlice to support more cases. Co-authored-by:
Mahesh Ravishankar <ravishankarm@google.com> Reviewed By: ThomasRaoux Differential Revision: https://reviews.llvm.org/D134294
-
Fangrui Song authored
In GNU ld, * --version skips linker input processing. * -v and -V keep processing if there is any input file. -V has more information we don't support. We currently make -V an alias for --version which skips input processing. On many `*-freebsd` and `powerpc-*` targets, `gcc -v` passes `-V` to ld and expects to process input. Make -V an alias for -v to provide compatibility. Fix https://github.com/llvm/llvm-project/issues/57859
-
Fangrui Song authored
-
Shubham Sandeep Rastogi authored
Differential Revision: https://reviews.llvm.org/D134290
-
Lei Zhang authored
Consecutive tensor.insert_slice/tensor.extract_slice can be created for the case like tiling convolution and then downsizing 2-D convolutions into 1-D ones. It hinders further transformations. So adding these patterns to clean it up. Given that bufferization is sensitive and have requirements over the IR structure (see https://reviews.llvm.org/D132666), these patterns are put in Transforms/ with separate entry points for explicit collection. Reviewed By: ThomasRaoux, mravishankar Differential Revision: https://reviews.llvm.org/D133871
-
Michael Jones authored
Strerror maps error numbers to strings. Additionally, a utility for mapping errors to strings was added so that it could be reused for perror and similar. Reviewed By: sivachandra Differential Revision: https://reviews.llvm.org/D134074
-
Jakub Kuderski authored
Suggested by @antiagainst in D134321.
-
Jakub Kuderski authored
This includes tests with the exact expected values and comparison-based tests. Reviewed By: antiagainst Differential Revision: https://reviews.llvm.org/D134321
-
Michael Berg authored
The type information of the store values can diverge when checking for valid mask store candidates to eliminate via DSE. This patch checks for equivalence wrt to size and element count. Reviewed By: fhahn, rui.zhang Differential Revision: https://reviews.llvm.org/D132700
-
Matheus Izvekov authored
Signed-off-by:
Matheus Izvekov <mizvekov@gmail.com> Differential Revision: https://reviews.llvm.org/D134295
-
Stanley Winata authored
Checks spirv::TargetEnv from op to see if it contains either Kernel or Shader capabilities. If it does, then it will set the memory space mapping accordingly. Reviewed By: antiagainst Differential Revision: https://reviews.llvm.org/D134317
-
Xiang Li authored
Add HLSLExternalSemaSource as ExternalSemaSource instead of ASTContext::ExternalSource when PCH is included. This allows a different external source to be set for the AST context. Reviewed By: beanz Differential Revision: https://reviews.llvm.org/D132421
-
Nirvedh Meshram authored
Reviwed By: antiagainst Differential Revision: https://reviews.llvm.org/D134217
-
Matthew Voss authored
This flag enables the .debug_aranges section by passing a flag to LLD and our internal linker. This also adds a new routine that will generate the correct flag for our internal linker or set of flags for LLD when given a list of LLVM options. That ensures multiple LLVM codegen options can be passed to either linker consistently. Differential Revision: https://reviews.llvm.org/D134296
-
Pengxuan Zheng authored
This patch adds support for link.exe's /MAPINFO flag to lld-link.exe. Here is a description of the flag from Microsoft (https://learn.microsoft.com/en-us/cpp/build/reference/mapinfo-include-information-in-mapfile?view=msvc-170): The /MAPINFO option tells the linker to include the specified information in a mapfile, which is created if you specify the /MAP option. EXPORTS tells the linker to include exported functions. Reviewed By: rnk Differential Revision: https://reviews.llvm.org/D134247
-
Lei Zhang authored
This commit adds a pattern to swap ``` tensor.extract_slice(linalg.fill(%cst, %init)) ``` into ``` linalg.fill(%cst, tensor.extract_slice(%init)) ``` when the linalg.fill op have no other users. This helps to reduce the fill footprint. Reviewed By: mravishankar Differential Revision: https://reviews.llvm.org/D134102
-
Stanley Winata authored
Templatizing vector reduction to enable lowering from vector.reduction max/min to CL ops. Reviewed By: antiagainst Differential Revision: https://reviews.llvm.org/D134313
-
Louis Dionne authored
-
Naje George authored
Reviewed By: ktras Differential Revision: https://reviews.llvm.org/D131535
-
Joshua Batista authored
remove abs non-elementwise attribute statements, stick to elementwise. Reviewed By: beanz Differential Revision: https://reviews.llvm.org/D134312
-
Kai Nacke authored
In `GIMatchTreeOpcodePartitioner::applyForPartition()`, the loop over the possible leaves skip a leaf if the instruction does not care about the instruction. When processing the referenced operands in the next loop the same leaves need to be skipped. Later, when these leaves are added to all partitions, the bit vector must be resized first before the bit representing the leaf is set. This fixes a crash in llvm-tblgen. Reviewed By: arsenm Differential Revision: https://reviews.llvm.org/D134192
-
Matt Arsenault authored
This was using the default address space.
-
Matt Arsenault authored
The expansion is currently using the wrong pointer size.
-
Matt Arsenault authored
-
Matt Arsenault authored
-