- Dec 07, 2023
-
-
Joseph Huber authored
Summary: This was added in a previous patch to update how we export the static library used for OpenMP offloading. By mistake this if-else was using the output incorrectly. Fixes https://github.com/llvm/llvm-project/issues/74079
-
Timm Bäder authored
-
W-50243 authored
'isctype' fails in arm64-big-endian because the __regex_word involved in mask operation is not changed based on the platform endianness, while the character mask does change.
-
Simon Pilgrim authored
-
Nhat Nguyen authored
Fixes #62433 Co-authored-by:Louis Dionne <ldionne.2@gmail.com>
-
Nikita Popov authored
When I originally added this fold, it did not actually fix my motivation case, where the add was represented as an or. Now that we have the disjoint flag this can finally be cleanly supported.
-
Nikita Popov authored
-
Utkarsh Saxena authored
-
Nikita Popov authored
This matches either a plain "add" or an "or disjoint" that can be converted into an add. The AddLike terminology is adopted from the SDAG layer.
-
Krzysztof Parzyszek authored
Replace explicit calls to ``` op = builder.create<SectionOp>(...) createBodyOfOp<SectionOp>(op, ...) ``` with a single call to ``` createOpWithBody<SectionOp>(...) ``` This is NFC, that's what the `createOpWithBody` function does.
-
Nikita Popov authored
This test contains an annoying mix of generated and hand-written check lines. Generate the whole test.
-
Aaron Ballman authored
This code was added 17 years ago but never enabled or tested. GCC warns that -I- is deprecated for them, and Clang gives an error when passed -I-, so we may as well remove this code rather than hook it up to the driver and maintain it.
-
Nikita Popov authored
I'm not sure whether it's possible to cause a miscompile due to the missing check right now, as the affected values mechanism effectively protects us against this. This becomes a problem for an upcoming patch though.
-
Shengchen Kan authored
-
wanglei authored
Mark ISD::SETCC node as legal, and add handling for the vector types condition codes.
-
Pablo Antonio Martinez authored
Test was failing due to a different transform sequence declaration (transform sequence were used, while now it should be named transform sequence). Test is now fixed.
-
Duo Wang authored
lld string tail merging interacts badly with ASAN on Windows, as is reported in https://github.com/llvm/llvm-project/issues/62078. A similar error was found when building LLVM with `-DLLVM_USE_SANITIZER=Address`: ```console [2/2] Building GenVT.inc... FAILED: include/llvm/CodeGen/GenVT.inc C:/Dev/llvm-project/Build_asan/include/llvm/CodeGen/GenVT.inc cmd.exe /C "cd /D C:\Dev\llvm-project\Build_asan && C:\Dev\llvm-project\Build_asan\bin\llvm-min-tblgen.exe -gen-vt -I C:/Dev/llvm-project/llvm/include/llvm/CodeGen -IC:/Dev/llvm-project/Build_asan/include -IC:/Dev/llvm-project/llvm/include C:/Dev/llvm-project/llvm/include/llvm/CodeGen/ValueTypes.td --write-if-changed -o include/llvm/CodeGen/GenVT.inc -d include/llvm/CodeGen/GenVT.inc.d" ================================================================= ==31944==ERROR: AddressSanitizer: global-buffer-overflow on address 0x7ff6cff80d20 at pc 0x7ff6cfcc7378 bp 0x00e8...
-
Michael Buch authored
-
Martin Storsjö authored
Whenever linking with -nodefaultlibs for a MinGW target, we manually need to specify a bunch of libraries - listed in ${MINGW_LIBRARIES}; the same is already done for sanitizers and libunwind/libcxxabi/libcxx. Practically speaking, linking with -nodefaultlibs but manually passing the libraries in ${MINGW_LIBRARIES} restores most of the libraries that are linked by default, except for the potential compiler builtins and unwind library; i.e. it has essentially the same effect as linking with "--unwindlib=none -rtlib=none", except that -rtlib doesn't accept such a value. When building only compiler-rt/lib/builtins, not all of compiler-rt, ${MINGW_LIBRARIES} is unset - set it manually here for that case. This matches what is set in compiler-rt/cmake/config-ix.cmake, except that the builtins (libgcc or compiler-rt builtins) is omitted; the only use within lib/buitlins is for the standalone libatomic, which explicitly already links against the just-built builtins. -
Mehdi Amini authored
Fixes #74611
-
Harald van Dijk authored
`llvm/test/CodeGen/RISCV/llvm.frexp.ll` and `llvm/test/CodeGen/X86/llvm.frexp.ll` contain a number of disabled tests for unimplemented functionality. This implements one missing part of it.
-
DonatNagyE authored
...that is causing the bug report when it's converted to the enum type. This commit only improves the diagnostics and does not affect the set of reports.
-
Tom Eccles authored
The implementation of these are modeled after the existing fastmath flags for floating point arithmetic.
-
Simon Pilgrim authored
If the target can generate a specific fp immediate constant, then don't split the store into 2 x i32 stores Another cleanup step for #74304
-
Jay Foad authored
-
Jay Foad authored
In GFX12 the exp instruction is renamed to export, but exp is still accepted as an alias. Co-authored-by:Mateja Marjanovic <mateja.marjanovic@amd.com>
-
Matt Arsenault authored
I don't believe this makes any practical difference. Fixes #64013
-
Nikita Popov authored
AST checks aliasing with MustAlias sets by only checking the representative pointer (getSomePointer). This is only correct if the Size and AATags information of that pointer also includes the Size/AATags of all other pointers in the set. When we add a new pointer to the AliasSet, we do perform this update (see the code in AliasSet::addPointer). However, if a pointer already in the MustAlias set is used with a new size, we currently do not update the representative pointer, resulting in miscompilations. Fix this by adding the missing update. This is a targeted fix using the current representation. There are a couple of alternatives: * For MustAlias sets, don't store per-pointer Size/AATags at all. This would make it clear that there is only one set of common Size/AATags for all pointers. * Check against all pointers in the set even for MustAlias. This is what https://github.com/llvm/llvm-project/pull/65731 proposes to do as part of a larger change to AST representation. Fixes https://github.com/llvm/llvm-project/issues/64897.
-
Mikhail Goncharov authored
This reverts commit c4399130. Test fails https://lab.llvm.org/buildbot/#/builders/272/builds/2757
-
Rik Huijzer authored
Fixes https://github.com/llvm/llvm-project/issues/62489. Some notes for each number: - 1 `bool-literal` should be reasonably clear from context. - 2 Fixed. - 3 This is now fixed. `loc(fused[])` is valid, but `loc(fused["foo",])` is not. - 4 This operation uses `assemblyFormat` so the syntax is correct (assuming ODS is correct). - 5 This operation uses `assemblyFormat` so the syntax is correct (assuming ODS is correct). - 6 Added an example. - 7 The suggested fix is in line with other `assemblyFormat` examples. - 8 Added syntax and an example. - 9 I don't know what this is referring too. - 10 Added example. - 11 and 12 suggestion seems wrong as the `ShapedTypeInterface` could be extended by clients, so is not limited to tensors or vectors. - 13 is already reasonably clear with the example, I think. - 14 is already reasonably clear with the example, I think. - 15 Added an example from the `opaque_locations.mlir` tests. - 16 The answer to this seems to change over time and depend on the use case? Suggestions by reviewers are welcome.
-
DonatNagyE authored
Eliminate the `mutable unique_ptr` hack because it's no longer needed. (This cleanup could be done anywhere, I'm doing it here now because it was me who published this checker with the old hack when it was already superfluous.)
-
Chuanqi Xu authored
[C++20] [Modules] Skip Writing diagnostic options, header search paths and pragma diagnostic mappings It simply wastes of space and time to write diagnostic options, header search paths and pragma diagnostic mappings for C++20 Named modules. This patch tries to avoid the unnecessary writings.
-
dong jianqiang authored
This patch propagates the -moutline flag when target is aarch64_be, fix warning: 'aarch64_be' does not support '-moutline'; flag ignored [-Woption-ignored]
-
Aiden Grossman authored
MAP_FIXED_NOREPLACE doesn't exist on older kernels, so we need to define it to be MAP_FIXED.
-
esmeyi authored
This PR adds the support for parsing auxiliary symbols of XCOFF object file for obj2yaml. The sanitizer error is clean now.
-
Pierre van Houtryve authored
-
Aiden Grossman authored
Currently, the llvm-exegesis LatencyBenchmarkRunner repeats the benchmark several times (currently 30) and then aggregates the result to deal with noise in the measurement process. With this patch, the number of repetitions to perform is made configurable rather than left as a static number. This allows for significantly faster execution in situations where someone is performing a task like experimenting with memory annotations where the exact cycle counts might not be useful, and also allows for increased precision when desired.
-
Sjoerd Meijer authored
This fixes a regression that occured for a pattern of MOVI + CMGT instructions, which can be optimised to CMGE. I.e., when the signed greater than compare has -1 as an operand, we can rewrite that as a compare greater equal than 0, which is what CMGE does. Fixes #61836
-
Pablo Antonio Martinez authored
This patchs adds the `filter_operand_types` attribute to transform::MatchOp, allowing to filter ops depending on their operand types.
-
Pierre van Houtryve authored
-