- May 21, 2022
-
-
Mitch Phillips authored
Needed for a TSan test that won't pass on PPC. Relevant information is in https://reviews.llvm.org/D110552.
-
Michael Kruse authored
Even though the comment description is ".unroll_inner.iv < NumIterations", the code emitted a BO_LE ('<=') operator for the inner loop that is to be unrolled. This lead to one additional copy of the body code in a partially unrolled. It only manifests when the unrolled loop is consumed by another loop-associated construct. Fix by using the BO_LT operator instead. The condition for the outer loop and the corresponding code for tiling correctly used BO_LT already. Fixes #55236 -
Aaron Ballman authored
We currently have a page for tracking defects against the C++ standard, but we don't have the same information for the C standard. This starts us down the path of being able to track that in a way our users can see. There are *a lot* of entries marked as "Unknown". As we validate Clang's behavior for a given DR by adding a test case for it, we can slowly begin to improve this page over time. This page is now linked from the existing C status page, which was updated slightly as a result of these changes as well. Note, unlike with the C++ defect report page, this content is not auto- generated from a source document and is not automatically updated from test comments. It may be worthwhile to automate the updates based on our test coverage, but that can happen later.
-
Mitch Phillips authored
Notably fails under PPC. For now, just exclude it. More details in the original Phabricator review, https://reviews.llvm.org/D110552.
-
Balazs Benics authored
The end-user has no way of 'fixing' bugs in the system library anyway. Let's suppress these as well. Reviewed By: LegalizeAdulthood Differential Revision: https://reviews.llvm.org/D125770
-
Balazs Benics authored
Unfortunately, we must restrict the checker to warn for deprecated headers only if the header is included directly from a c++ source file. For header files, we cannot know if the project has a C source file that also directly/indirectly includes the offending header file otherwise. Thus, it's better to be on the safe side and suppress those reports. One can opt-in the old behavior, emitting diagnostics into header files, if one explicitly sets the WarnIntoHeaders=true, in which case nothing will be changed. Reviewed By: LegalizeAdulthood Differential Revision: https://reviews.llvm.org/D125769
-
Joseph Huber authored
Summary: We usually used the `OMP_LIKELY` and `OMP_UNLIKELY` macros to add branch prediction intrinsics to help the optimizer ignore unlikely loops. This wasn't applied to this one loop so add that in.
-
Balazs Benics authored
This partially reverts commit e8cae487. Changes since that commit: - Use `SourceManager::isBeforeInTranslationUnit` instead of the fancy decomposed decl logarithmic search. - Add a test for including a system header containing a deprecated include. - Add `REQUIRES: system-linux` clause to the test. Reviewed By: LegalizeAdulthood, whisperity Differential Revision: https://reviews.llvm.org/D125209
-
Zequan Wu authored
It saves about 1.13% size for chrome.dll.pdb on chrome official build. Reviewed By: rnk Differential Revision: https://reviews.llvm.org/D125721
-
Bill Wendling authored
The FPR128 regs need MOVIv2d_ns and SVE regs need DUP_ZI_D. Differential Revision: https://reviews.llvm.org/D126083
-
Alexander Shaposhnikov authored
This recommits https://reviews.llvm.org/rG6990e7477d24ff585ae86549f5280f0be65422a6 as the problematic test has been updated updated in https://reviews.llvm.org/rG3bd112c720dc614a59e3f34ebf9b45075037bfa0.
-
Jim Ingham authored
The test for commit bff4673b is failing on the GreenDragon bot but none of us can repro the failure locally. Adding some logging to the test failure to help diagnose the issue.
-
Sam McCall authored
-
Mitch Phillips authored
3bd112c7 fixed the fuzzing test on Linux, which, after https://reviews.llvm.org/D125933, has one less branch. Turns out, on Windows, that it still has the extra branch. I'm guessing that's because exit() isn't known to be noreturn on Windows or something. Either way, just make the test more tolerant.
-
Adrian Prantl authored
https://reviews.llvm.org/D125496 changed the layout of std::string without updating the LLDB dataformatter. This patch adds code to recognize the new format. Differential Revision: https://reviews.llvm.org/D126080
-
Mitch Phillips authored
https://reviews.llvm.org/D125933 improved some of LLVM's handling of binary ORs, which meant we have one less conditional branch, because the 'if (Size > 5 && Data[5] == 'R')' and 'if (bits == 63)' branches are now correctly folded.
-
Douglas Yung authored
This reverts commit 6990e747. This change was causing the test compiler-rt/test/fuzzer/merge_two_step.test to fail on our internal bot as well as other build bots such as https://lab.llvm.org/buildbot/#/builders/179/builds/3712.
-
Craig Topper authored
-
Philip Reames authored
Establish the most basic possible test coverage for LSR transformation on RISCV. Original patch by eopXD (D123458), modified by me to cleanup/simplify tests.
-
Nico Weber authored
-
Christopher Bate authored
The missing link dependency caused build failures in some configurations.
-
Jonas Devlieghere authored
Report the correct register number (GENERIC_REGNUM_FLAGS) for cpsr. This fixes TestLldbGdbServer.py on Apple Silicon. Differential revision: https://reviews.llvm.org/D126076
-
Craig Topper authored
If the SafeWrap operation is a subtract, we negated the constant to treat the subtract as an addition. The sext was based on the operation being addition. So we really need to do (neg (sext (neg C))) when promoting the constant. This is equivalent to (sext C) for every value of C except the min signed value. For min signed value we need to do (zext C) instead. Fixes PR55490. Differential Revision: https://reviews.llvm.org/D125653
-
Qiongsi Wu authored
https://reviews.llvm.org/D123498 contains a few errors resulting in incorrect target contents or mismatched target/list names. This patch fixes all the known errors. Reviewed By: craig.topper Differential Revision: https://reviews.llvm.org/D126002
-
Jay Foad authored
This brings the MachineInstrs in line with the corresponding intrinsics which have side effects but do not access memory. It also matches how BUF cache invalidation instructions are defined. The lit test changes are just because the machine scheduler previously treated them like loads, and added an artificial scheduling edge from them to the exit SU, which caused them to be scheduled earlier. Differential Revision: https://reviews.llvm.org/D126074
-
Dmitri Gribenko authored
-
Dmitri Gribenko authored
-
Dmitri Gribenko authored
-
- May 20, 2022
-
-
Aart Bik authored
Reviewed By: bixia Differential Revision: https://reviews.llvm.org/D126039
-
David Goldman authored
Previously, clangd would filter completions only on the first part of the selector (first typed chunk) instead of all remaining selector fragments (all typed chunks). Differential Revision: https://reviews.llvm.org/D124637
-
Christopher Bate authored
This changes adds the option to lower to NvGpu dialect ops during the VectorToGPU convsersion pass. Because this transformation reuses existing VectorToGPU logic, a seperate VectorToNvGpu conversion pass is not created. The option `use-nvgpu` is added to the VectorToGPU pass. When this is true, the pass will attempt to convert slices rooted at `vector.contract` operations into `nvgpu.mma.sync` ops, and `vector.transfer_read` ops are converted to either `nvgpu.ldmatrix` or one or more `vector.load` operations. The specific data loaded will depend on the thread id within a subgroup (warp). These index calculations depend on data type and shape of the MMA op according to the downstream PTX specification. The code for supporting these details is separated into `NvGpuSupport.cpp|h`. Differential Revision: https://reviews.llvm.org/D122940
-
Alex Brachet authored
Differential revision: https://reviews.llvm.org/D125658
-
Nabeel Omer authored
Introduces basic test coverage for frem on x86. Split off from https://reviews.llvm.org/D125988 Differential Revision: https://reviews.llvm.org/D126055
-
Jonas Devlieghere authored
Disable scripted_crashlog_json.test on Apple Silicon until Ismail has bandwidth to investigate. rdar://93655633
-
Jay Foad authored
Extend SIInstrInfo::isOperandLegal to enforce a limit on the number of literal operands for all VALU instructions, not just VOP3. In particular it now handles VOP2 instructions with a mandatory literal operand like V_FMAAK_F32. Differential Revision: https://reviews.llvm.org/D126064
-
Jay Foad authored
Extend the literal operand checking in SIInstrInfo::verifyInstruction to check VOP2 instructions like V_FMAAK_F32 which have a mandatory literal operand. The rule is that src0 can also be a literal, but only if it is the same literal value. AMDGPUAsmParser::validateConstantBusLimitations already handles this correctly. Differential Revision: https://reviews.llvm.org/D126063
-
Dmitri Gribenko authored
-
Nikolas Klauser authored
Reviewed By: ldionne, #libc Spies: jwakely, rodgert, libcxx-commits Differential Revision: https://reviews.llvm.org/D125634
-
Guillaume Chatelet authored
This reverts commit 94d6dd90.
-
Sam McCall authored
LSP supports Diagnostic.codeInformation since 3.16. In VSCode, this turns the code (e.g. "unused-includes" or "bugprone-foo") into a clickable link that opens the docs in a web browser. Differential Revision: https://reviews.llvm.org/D126065
-