- Mar 09, 2023
-
-
Nikita Popov authored
This reverts commit d14e7ee3. Makes clang_format_diff.py always fail with: Traceback (most recent call last): File "/home/npopov/repos/llvm-project/clang/tools/clang-format/clang-format-diff.py", line 177, in <module> main() File "/home/npopov/repos/llvm-project/clang/tools/clang-format/clang-format-diff.py", line 174, in main process_subprocess_result(proc, args) File "/home/npopov/repos/llvm-project/clang/tools/clang-format/clang-format-diff.py", line 42, in process_subprocess_result with open(filename) as f: ^^^^^^^^ NameError: name 'filename' is not defined
-
Markus Mützel authored
When linking a shared library with Flang on MinGW, the functions from the Flang runtime are exported from the shared library. When trying to link an executable to that library using Flang, the linker errors out because the functions from the runtime conflict with the functions exported from the shared library. Add the Flang runtime libraries to the list of libraries for which no symbols are exported. Reapplying the patch with the git author name corrected. Reviewed By: mstorsjo Differential Revision: https://reviews.llvm.org/D145389
-
Martin Storsjö authored
This reverts commit 04fc67bf. The patch was applied with the wrong git author name (forgot to amend in the right user name after applying the patch); I'll reapply it with the right author name.
-
Simon Pilgrim authored
-
Martin Storsjö authored
When linking a shared library with Flang on MinGW, the functions from the Flang runtime are exported from the shared library. When trying to link an executable to that library using Flang, the linker errors out because the functions from the runtime conflict with the functions exported from the shared library. Add the Flang runtime libraries to the list of libraries for which no symbols are exported. Reviewed By: mstorsjo Differential Revision: https://reviews.llvm.org/D145389
-
ManuelJBrito authored
Ignoring freeze(undef) if it has multiple uses in LowerAVXCONCAT_VECTORS causes the custom INSERT_SUBVECTOR for vector widening to be ignored. Differential Revision: https://reviews.llvm.org/D144903
-
Balázs Kéri authored
The code was difficult to maintain (big internal class definitions with long inline functions, other functions of the same class at different location far away, irregular ordering of classes and function definitions). It is now improved to some extent. New functions are added to RangeConstraint to remove code repetition, these are useful for planned new features too. Comments are improved. Reviewed By: Szelethus Differential Revision: https://reviews.llvm.org/D143751
-
Haojian Wu authored
-
Sven van Haastregt authored
Add an emitter to produce something similar to opencl-c.h from the OpenCL builtin descriptions in OpenCLBuiltins.td This only adds the emitter, without any direct use of it. This allows opencl-c.h additions to be generated from the builtin descriptions by manually invoking `clang-tblgen -gen-clang-opencl-builtin-header`. Differential Revision: https://reviews.llvm.org/D104040
-
Petar Avramovic authored
Values in SGPR and VGPR register are treated as unsigned by hardware. When value in 32-bit SGPR or VGPR base can be negative calculate offset using 32-bit add instructions, otherwise use sgpr(unsigned) + vgpr(unsigned) + offset. LoopStrengthReduce.cpp changes offsets to negative and in some iterations value in SGPR or VGPR register could be negative. Differential Revision: https://reviews.llvm.org/D144957
-
Petar Avramovic authored
Values in VGPR register are treated as unsigned by hardware. When value in 32-bit VGPR base can be negative calculate offset using 32-bit add instruction, otherwise use vgpr base(unsigned) + offset. Does not affect case where whole offset comes from VGPR register (immediate offset is 0). LoopStrengthReduce.cpp changes offsets to negative and in some iterations value in VGPR register could be negative. Differential Revision: https://reviews.llvm.org/D144956
-
Petar Avramovic authored
Values in SGPR register are treated as unsigned by hardware. When value in 32-bit SGPR base can be negative calculate offset using 32-bit add instruction, otherwise use sgpr base(unsigned) + offset. Does not affect case where whole offset comes from SGPR register (immediate offset is 0). LoopStrengthReduce.cpp changes offsets to negative and in some iterations value in SGPR register could be negative. Differential Revision: https://reviews.llvm.org/D144955
-
Yeting Kuo authored
[IR][DAG][RISCV] Allow scalable vector ISD::STRICT_FP_EXTEND and RISC-V supports for vector ISD::STRICT_FP_EXTEND. The patch mainly does two things. The first is allowing scalable vector ISD::STRICT_FP_EXTEND. The second is making RISC-V customized lower strict_fpextend to riscv_strict_fpextend_vl, the strict version of riscv_fpextend_vl. Reviewed By: craig.topper Differential Revision: https://reviews.llvm.org/D145548
-
Chuanqi Xu authored
Close https://github.com/llvm/llvm-project/issues/61065. We will avoid writing the names from external AST naturally. But currently its check is often false positive since we may have already marked the declarations as external but DeclContext::hasNeedToReconcileExternalVisibleStorage would be false after reconciling. Tested with libcxx's modular build. This patch can improve 8% compilation time in an internal workloads.
-
Nikita Popov authored
Since https://reviews.llvm.org/D141386 !range violations return poison instead of causing immediate undefined behavior. As such, it is fine for IPSCCP to infer !range even if the value might be poison. (The value cannot be undef as this would promote undef to poison, but this is already checked separately.) This basically undoes the late change done to D83952, restoring it to its original version (which is now valid). Differential Revision: https://reviews.llvm.org/D144467
-
Nikita Popov authored
Use the memory() spelling in a few places that were using the old syntax. The documented attributes for llvm.type.checked.load don't match the actual attributes, I've raised this here: https://reviews.llvm.org/D21121#inline-1406792
-
Han Zhu authored
Fix a crash when compiling Skia. See https://reviews.llvm.org/D143225#4180342 for more details
-
Marco Elver authored
SanitizerBinaryMetadata should only apply to to host code, and not GPU code. Recently AMD GPU target code has experimental sanitizer support. If we're compiling a mixed host/device source file, only add sanitizer metadata to host code. Differential Revision: https://reviews.llvm.org/D145519
-
David Spickett authored
-
Nikita Popov authored
The legacy PM is only supported for codegen, and PassManagerBuilder is exclusively about the middle-end optimization pipeline. Drop it. Differential Revision: https://reviews.llvm.org/D145387
-
Siva Chandra Reddy authored
-
Haojian Wu authored
Differential Revision: https://reviews.llvm.org/D145557
-
Haojian Wu authored
Differential Revision: https://reviews.llvm.org/D145553
-
Chuanqi Xu authored
The option '-fmodule-file=<module-name>=<path-BMI>' is already deprecated now. This patch mentions it in the document.
-
Siva Chandra Reddy authored
The libc runtime code is already compiled with -fpie.
-
Aaron Siddhartha Mondal authored
These libraries are only ever used in clang-tidy itself, so there is no need to unconditionally keep all symbols. Reviewed By: GMNGeoffrey, #bazel_build Differential Revision: https://reviews.llvm.org/D145258
-
Karl-Johan Karlsson authored
When compiling compiler-rt with -fsanitize=undefined and running testcases you end up with the following warning: UBSan: int_mulo_impl.inc:21:36: left shift of 1 by 63 places cannot be represented in type 'di_int' (aka 'long long') This can be avoided by simply doing the shift in a matching unsigned variant of the type. The same kind of pattern seems to exist in int_mulv_impl.inc This was found in an out of tree target. Reviewed By: MaskRay Differential Revision: https://reviews.llvm.org/D145556
-
Siva Chandra Reddy authored
-
Théo Degioanni authored
This revision adds the inalloca attribute to the alloca operation in the LLVMIR dialect. It also adds tests for import and export. Reviewed By: gysit Differential Revision: https://reviews.llvm.org/D145483
-
Piyou Chen authored
This commit enable the subregister liveness by default in RISC-V. It was previously disabled in https://reviews.llvm.org/D129646 after a previous attempt to enabled it https://reviews.llvm.org/D128016. We believe that https://reviews.llvm.org/D129735 fixes the issue that caused it to be disabled. Reviewed By: craig.topper, kito-cheng Differential Revision: https://reviews.llvm.org/D145546
-
Craig Topper authored
fli.h requires Zfh or Zvfh. We need to check for this in isFPImmLegal. Zvfh support will come in another patch. I had to split the test file because there are other issues with Zfhmin and some intrinsics.
-
Bing1 Yu authored
Update X86MemFoldTables.inc according to https://reviews.llvm.org/D142084's auto-gen table Reviewed By: skan Differential Revision: https://reviews.llvm.org/D143149
-
Igor Zhukov authored
Reviewed By: #libc Differential Revision: https://reviews.llvm.org/D145651
-
Jonas Devlieghere authored
Fixed warning: suggest explicit braces to avoid ambiguous ‘else’ [-Wdangling-else]
-
Jonas Devlieghere authored
-
Jonas Devlieghere authored
Fix the crash in SymbolVendorELF and re-enable the test.
-
Craig Topper authored
This was added after the patch was approved. I'm not sure why its there. It doesn't fire in any lit test.
-
Craig Topper authored
-
Jonas Devlieghere authored
The new test is triggering a crash in LLDB on the Windows and Linux bots. Temporarily disable the test while I investigate.
-
Jie Fu authored
/home/jiefu/llvm-project/polly/lib/CodeGen/IslNodeBuilder.cpp:80:11: error: unused variable 'VectorLoops' [-Werror,-Wunused-variable] STATISTIC(VectorLoops, "Number of generated vector for-loops"); ^ 1 error generated.
-