- May 18, 2023
-
-
Vitaly Buka authored
Msan complains if getFoundDecl called before it's set. This looks like data class, so I see no reason to keep the single field uninitialized.
-
Vitaly Buka authored
-
Vitaly Buka authored
I assume it's optional and ReadAST does not have to set the counter on success. Without the patch msan complains that we pass uninitialized value into noundef parameters of setCounterValue. Reviewed By: bnbarham, barannikov88 Differential Revision: https://reviews.llvm.org/D150492
-
Chuanqi Xu authored
Currently, the header units are rarely used and it is not well tested. To avoid further misunderstandings, let's mark it as experimental and emit a warning when users wants to import it. This is discussed in modules developers meeting.
-
Vitaly Buka authored
Fixes false leaks on thread retval. Reviewed By: thurston Differential Revision: https://reviews.llvm.org/D150165
-
Vitaly Buka authored
Fixes false leaks on thread retval. Reviewed By: thurston Differential Revision: https://reviews.llvm.org/D150106
-
Vitaly Buka authored
Luckely of Linux PTHREAD_CREATE_DETACHED is 1.
-
Matt Arsenault authored
InstCombine already does this for minnum/maxnum. If we also apply this to fmed3, we don't need to explicitly use 16-bit fmed3 if we're not sure the target supports 16-bit instructions yet.
-
Matt Arsenault authored
-
Matt Arsenault authored
-
Fangrui Song authored
This restores 737a4521 (2013) which was removed by my a07b135c. This behavior turns out to be depended on by `bazel coverage`. Add a `PWD=/proc/self/cwd %clang -### -c --coverage %s -o foo/bar.o` test.
-
Vitaly Buka authored
Reproducible with =-1 and assert: https://reviews.llvm.org/P8309 Reviewed By: jmorse Differential Revision: https://reviews.llvm.org/D150420
-
Wang, Xin10 authored
Missing work for D150835, aside from VMASKMOVDQU, MASKMOVDQU will also be affected by D150436. Reviewed By: skan Differential Revision: https://reviews.llvm.org/D150844
-
Matt Arsenault authored
Doesn't try the all the tricks computeKnownBits does.
-
Matt Arsenault authored
Allows assume handling to work.
-
Matt Arsenault authored
This should now be redundant with the nan handling in computeKnownFPClass.
-
Matt Arsenault authored
computeKnownFPClass should now handle infinity checks equally as well as what this could do before so delete the redundant code.
-
Jianjian GUAN authored
Use AllVectors to replace !listconcat(AllIntegerVectors, AllFloatVectors). Reviewed By: craig.topper Differential Revision: https://reviews.llvm.org/D150837
-
Craig Topper authored
-
Craig Topper authored
This was copied from SVE, but is currently not applicable to RISC-V.
-
Aiden Grossman authored
This reverts commit 286cefcf. Patch caused build failures for downstream projects on Windows due to the fact that __cpuidex was added as a built in on Windows in D121653. Reverting for now so that others aren't blocked and I can figure out a proper solution.
-
Mehdi Amini authored
The properties size is compressed as a member of the Operation class to assume a multiple of 8B is used for the storage. This matched the natural alignment / padding on 64 bits platforms, however we need some explicit padding on 32 bits platforms, llvm::TrailingObjects will compress and misalign. Fixes #62763
-
Amir Ayupov authored
Make intent more explicit with the use of new helper methods. Reviewed By: #bolt, maksfb Differential Revision: https://reviews.llvm.org/D150810
-
Carl Ritson authored
Mark V_MBCNT instructions add thread/lane position so will never be uniform. Reviewed By: foad Differential Revision: https://reviews.llvm.org/D150759
-
Joshua Cao authored
turnGuardIntoBranch() can use splitBlockAndInsertIfThen to update the DominatorTree rather than implementing it itself.
-
Craig Topper authored
Reviewed By: eopXD Differential Revision: https://reviews.llvm.org/D150690
-
Kazu Hirata authored
-
Kazu Hirata authored
While we are at it, this patch removes unnecessary includes.
-
Kazu Hirata authored
The corresponding function definition was removed by: commit 9071393c Author: Jay Foad <jay.foad@amd.com> Date: Thu Feb 17 14:17:36 2022 +0000
-
Wang, Xin10 authored
VMASKMOVDQU supports 32bit/64bit version in 64bitmode, previously we prefer to use VMASKMOVDQU64 in 64bitmode because the 32bit one need 0x67 prefix. After D150436, asm match table changed a little, which makes in 64bit mode "vmaskmovdqu %xmm0, %xmm1" will match VMASKMOVDQU other than VMASKMOVDQU64, this patch correct the asm match order for this instruction. Reviewed By: skan Differential Revision: https://reviews.llvm.org/D150835
-
Kazu Hirata authored
The last use was removed by: commit e98944ed Author: Stanislav Mekhanoshin <Stanislav.Mekhanoshin@amd.com> Date: Mon Mar 11 17:04:35 2019 +0000
-
Shengchen Kan authored
-
Emilia Kond authored
When in ColumnLimit 0, the formatter looks for MustBreakBefore in the line in order to check if a line is allowed to be merged onto one line. However, since MustBreakBefore is really a property of the gap between the token and the one previously, I belive the check is erroneous in checking all the tokens in a line, since whether the previous line ended with a forced line break should have no effect on whether the current line is allowed to merge with the next one. This patch changes the check to skip the first token in `LineJoiner.containsMustBreak`. This patch also changes a test, which is not ideal, but I believe the test also suffered from this bug. The test case in question sets AllowShortFunctionsOnASingleLine to "Empty", but the empty function in said test case isn't merged to a single line, because of the very same bug this patch fixes. Fixes https://github.com/llvm/llvm-project/issues/62721 Reviewed By: HazardyKnusperkeks, owenpan, MyDe...
-
Katherine Rasmussen authored
Apply the check for the constraint `C1172` to `unlock-stmt`, `change-team-stmt`, `end-team-stmt`, and `critical-stmt`, which all have `sync-stat-lists` and so `C1172` applies to them. Add a test to check the `sync-stat-lists` for these 4 statements. Reviewed By: PeteSteinfeld Differential Revision: https://reviews.llvm.org/D150745
-
Jianjian GUAN authored
This patch is similar to https://reviews.llvm.org/D150550, it adds accurate predicates for SDNode patterns depending on vector type. Reviewed By: craig.topper Differential Revision: https://reviews.llvm.org/D150754
-
Sergei Barannikov authored
Preparatory change for D148094.
-
Alex Langford authored
We should specify that this is the pid_t as defined in the lldb namespace, not some other pid_t. This doesn't really affect builds but it makes writing tooling against the SBAPI easier. I have verified that this does not change the emitted mangled name and does not break ABI.
-
wren romano authored
For some reason, even though D150822 passed the buildbot, it failed to catch this test Reviewed By: anlunx Differential Revision: https://reviews.llvm.org/D150830
-
Jan Svoboda authored
-
Fangrui Song authored
After a07b135c, we always pass -coverage-notes-file/-coverage-data-file for driver options -ftest-coverage/-fprofile-arcs/--coverage. As a bonus, we can make the following simplification to cc1 options: * `-ftest-coverage -coverage-notes-file a.gcno` => `-coverage-notes-file a.gcno` * `-fprofile-arcs -coverage-data-file a.gcda` => `-coverage-data-file a.gcda` and remove EmitCovNotes/EmitCovArcs.
-