- Sep 14, 2023
-
-
Dmitry Ilvokhin authored
This fixes some unused variable and "comparison of integers of different signs" warnings in the benchmarks. Differential Revision: https://reviews.llvm.org/D156613
-
Peter Klausler authored
The -fdebug-dump-all flag invokes runtime type information generation even for a program with fatal semantic errors. This could cause a crash on a failed CHECK(), since the type information table generator assumes a correct program. Make it more resilient for a known fatal case. (But if we hit many more of these, we should look into not generating the runtime type information tables under this flag.)
-
Nick Desaulniers authored
I recently went to merge a PR that had a merge conflict: $ gh pr merge --squash --delete-branch X Pull request #66003 is not mergeable: the merge commit cannot be cleanly created. To have the pull request merged after all the requirements have been met, add the `--auto` flag. Run the following to resolve the merge conflicts locally: gh pr checkout 66003 && git fetch origin main && git merge origin/main This is how I resolved it; we should recommend this explicitly for fellow contributors. -
Jan Svoboda authored
-
Michael Jones authored
The two decimal float printing styles are similar, but different in how they end. For simplicity of writing I initially gave them different "write_last_block" functions. This patch unifies them into one function. Reviewed By: sivachandra Differential Revision: https://reviews.llvm.org/D158036
-
Noah Goldstein authored
Reviewed By: RKSimon Differential Revision: https://reviews.llvm.org/D159325
-
Noah Goldstein authored
eq/ne are generally easier to reason about elsewhere. ult -> ne: https://alive2.llvm.org/ce/z/5wxXGt uge -> eq: https://alive2.llvm.org/ce/z/Dw6kqG Reviewed By: nikic Differential Revision: https://reviews.llvm.org/D145425
-
Noah Goldstein authored
Reviewed By: RKSimon Differential Revision: https://reviews.llvm.org/D145424
-
Arthur Eubanks authored
-
Michael Jones authored
In the document on undefined behavior, I noted that writing down your decisions is very important. This document contains all the information for compile flags and undefined behavior for our printf. Reviewed By: sivachandra Differential Revision: https://reviews.llvm.org/D158311
-
Jan Svoboda authored
-
Nick Desaulniers authored
reland [InlineAsm] wrap ConstraintCode in enum class NFC (#66003) This reverts commit ee643b70. Fix up build failures in targets I missed in #66003 Kept as 3 commits for reviewers to see better what's changed. Will squash when merging. - reland [InlineAsm] wrap ConstraintCode in enum class NFC (#66003) - fix all the targets I missed in #66003 - fix off by one found by llvm/test/CodeGen/SystemZ/inline-asm-addr.ll
-
Louis Dionne authored
Thanks @Gadal for noticing. Fixes #63703
-
Maryam Moghadas authored
This patch reverts commit 7614ba0a to optimize VPERM when one of its vector operands is XXSWAPD, similar to XXPERM. It also reorganizes the little-endian swap code on LE, swapping the vector operand after adjusting the mask operand. This ensures that the vector operand is swapped at the correct point in the code, resulting in a valid constant pool for the mask operand. Reviewed By: stefanp Differential Revision: https://reviews.llvm.org/D149083
-
Jakub Kuderski authored
This is a cleanup in preparation for adding a second conversion path using the KHR cooperative matrix extension. Make the existing lowering explicit about emitting ops from the NV coop matrix extension. Clean up surrounding code.
-
Joseph Huber authored
Summary: This comment is outddated and can be removed, also mention an option for limiting parallelism during tests in the documentation.
-
Jordan Rupprecht authored
These were added in 2011 in 54da06ca and 0d901121. It was then incompletely removed in 2015 in c268835e: the video itself was removed, but the VideoJS css & script is still loaded.
-
Paul T Robinson authored
-
Terry Wilmarth authored
Fix /tmp approach, and add environment variable method as third fallback during library registration The /tmp fallback for /dev/shm did not write to a fixed filename, so multiple instances of the runtime would not be able to detect each other. Now, we create the /tmp file in much the same way as the /dev/shm file was created, since mkstemp approach would not work to create a file that other instances of the runtime would detect. Also, add the environment variable method as a third fallback to /dev/shm and /tmp for library registration, as some systems do not have either. Also, add ability to fallback to a subsequent method should a failure occur during any part of the registration process. When unregistering, it is assumed that the method chosen during registration should work, so errors at that point are ignored. This also avoids a problem with multiple threads trying to unregister the library.
-
Peiming Liu authored
Many frontends canonicalize scalar into 0-ranked tensor, it change will hopefully make the operation easier to use for those cases.
-
Jeffrey Byrnes authored
Adds test to show impact on cross block CopyToReg & CopyFromReg handling for n x i8, and shows NFC on CC Differential Revision: https://reviews.llvm.org/D159303 Change-Id: Ib6d9802dbebe8e3245e4ccfd4a6f23357de8c480
-
Justin Bogner authored
This adds more validation that a dxil triple is actually useable when compiling HLSL. The OS field of the triple needs to be a versioned shader model. Later, we should set a default if this is empty and check that the version is a shader model we can actually handle. The Environment field of the triple needs to be specified and be a valid shader stage. I'd like to allow this to be empty and treat it like library, but allowing that currently crashes in DXIL metadata handling. Differential Revision: https://reviews.llvm.org/D159103
-
Vitaly Buka authored
-
michaelrj-google authored
This patch adds the long double table option for printf into the new configuration scheme. This allows it to be set for most targets but unset for baremetal.
-
chbessonova authored
Use symbol's flags saved in `NMSymbol::SymFlags` inside `NMSymbol::isDefined()` since `BasicSymbolRef::getFlags()` requires the symbol's containing entity object to exist (which doesn't, causing llvm-nm to crash).
-
Philip Reames authored
No need to special case add 0, N. SelectionDAG::getNode contains the canonicalization and simplification for this case, so no need to duplicate it here.
-
Vitaly Buka authored
LLVMSymbolizer::DemangleName returns the same input if it can't demangle. We can't tell if this is already demangled or format is unsupported. Internally DemangleName uses nonMicrosoftDemangle which can report a failure.
-
Yitzhak Mandelbaum authored
[clang][dataflow] Change `diagnoseFunction` to use `llvm::SmallVector` instead of `std::vector`. (#66014) The template is agnostic as to the type used by the list, as long as it is compatible with `llvm::move` and `std::back_inserter`. In practice, we've encountered analyses which use different types (`llvm::SmallVector` vs `std::vector`), so it seems preferable to leave this open to the caller.
-
Joseph Huber authored
Summary: This patch simply adds the necessary config to enable qsort and bsearch on the GPU. It is *highly* unlikely that anyone will use these, as they are single threaded, but we may as well support all entrypoints that we can.
-
Stefan Gränitz authored
-
Stefan Gränitz authored
-
Stefan Gränitz authored
We don't have to set or clear the Thumb bit in relocation fixup values. It's not part of the branch range and the respective encoding functions like encodeImmBT4BlT1BlxT2() shift out the least significant bit anyway. This was a leftover from the initial patch before we switched to store Thumb state in target-flags with D146641.
-
Reid Kleckner authored
This reverts commit 2ca4d136. Also revert the followup, "[InlineAsm] fix botched merge conflict resolution" This reverts commit 8b9bf3a9. There were SystemZ and Mips build errors, too many to fix forward.
-
Luke Lau authored
-
Louis Dionne authored
This partially reverts commit e30a148b, which removed the base template for std::char_traits. That base template had been marked as deprecated since LLVM 16 and we were planning to remove it in LLVM 18. However, as explained in the post-commit comments in https://reviews.llvm.org/D157058, the deprecation mechanism didn't work as expected. Basically, the deprecation warnings were never shown to users since libc++ headers are system headers and Clang doesn't show warnings in system headers. As a result, this removal came with basically no lead time as far as users are concerned, which is a poor experience. For this reason, I am re-introducing the deprecated char_traits specialization until we have a proper way of phasing it out in a way that is not a surprise for users.
-
Philip Reames authored
A splat index means the operation is reading from (writing to) the same memory location. Generally, zero is the cheapest value to splat. As such, we'd prefer to add the splatted value to the base, and use a constant zero as the index operand.
-
Simon Pilgrim authored
If we know the condition mask sign bit of the demanded elements then we can directly return the LHS/RHS selection
-
frgossen authored
-
Nick Desaulniers authored
Link: #66003