- Feb 12, 2023
-
-
Lang Hames authored
Commit 10b5fec2 uses a char -1 value, which triggers narrowing conversion warnings / errors on some platforms (see e.g. https://lab.llvm.org/buildbot/#/builders/36/builds/30312). Adding an explicit conversion should fix this.
-
Hui authored
Differential Revision: https://reviews.llvm.org/D143828
-
Craig Topper authored
-
Lang Hames authored
Renames the existing allocateString method to allocateContent and adds a pair of allocateCString methods. The previous allocateString method did not include a null-terminator. It behaved the same as allocateContent except with a Twine input, rather than an ArrayRef<char>. Renaming allocateString to allocateBuffer (overloading the existing method) makes this clearer. The new allocateCString methods allocate the given content plus a null-terminator character, and return a buffer covering both the string and null-terminator. This makes them suitable for creating c-string content for jitlink::Blocks. Existing users of the old allocateString method have been updated to use the new allocateContent overload.
-
Hui authored
Fix several bugs: 1. https://llvm.org/PR60258 The conversion constructors' constraint `__compatible_with` incorrectly allow array types conversion to scalar types 2. https://llvm.org/PR53368 The constructor that takes `unique_ptr` are not suffiently constrained. 3. The constructors that take raw pointers incorretly use `__compatible_with`. They have different constraints Differential Revision: https://reviews.llvm.org/D143346
-
Simon Pilgrim authored
getFauxShuffleMask can't handle ISD::TRUNCATE itself as it can't handle inputs that are larger than the output Another step towards removing combineX86ShuffleChainWithExtract
-
Nikolas Klauser authored
Fixes #60658 Reviewed By: Mordante, #libc Spies: aheejin, sstefan1, libcxx-commits Differential Revision: https://reviews.llvm.org/D143737
-
LLVM GN Syncbot authored
-
Lang Hames authored
-
Lang Hames authored
This allows these details to be shared with JITLink, which is allowed to depend on the OrcShared library (but not on OrcJIT).
-
NAKAMURA Takumi authored
-
Louis Dionne authored
This makes it much easier to figure out what is and what isn't implemented.
-
NAKAMURA Takumi authored
Its user may include its headers as `GlobalISel/*.h`
-
Louis Dionne authored
Those tests are arguably not great anyways, but it's better to have then as .verify tests than compile failure tests, which are too brittle.
-
Nikolas Klauser authored
All supported compilers support `consteval`, so there is no more need for the macro. Reviewed By: ldionne, Mordante, #libc Spies: libcxx-commits Differential Revision: https://reviews.llvm.org/D143489
-
Louis Dionne authored
-
Louis Dionne authored
-
Louis Dionne authored
Only the actual inheritance needs to be guarded.
-
Louis Dionne authored
-
Louis Dionne authored
-
Louis Dionne authored
-
- Feb 11, 2023
-
-
Tue Ly authored
-
Hui authored
move all range iterators back in class, as out of class iterators requires extra template parameters, which changes ADL Differential Revision: https://reviews.llvm.org/D143324
-
Hui authored
-
Markus Böck authored
-
Simon Pilgrim authored
These will be reused in a future patch
-
Simon Pilgrim authored
combineX86ShuffleChain and combineX86ShuffleChainWithExtract no longer require the shuffle inputs to be the same width as the root vector, so we can stop generating widening nodes on the fly (combineX86ShuffleChain should handles all of this). This requires a couple of additional folds to avoid a couple of notable regressions: getFauxShuffleMask - recognise INSERT_SUBVECTOR(X,Y,C) as a shuffle pattern as long as its not just widening the subvector. combineConcatVectorOps - folds CONCAT_VECTORS(AssertSext(X,Ty),AssertSext(Y,Ty)) -> AssertSext(CONCAT_VECTORS(X,Y),Ty) One of the final stages towards fixing Issue #45319 and addressing the regressions in the interleaved tests in D127115
-
Tobias Gysi authored
Update the FuncToLLVM pass to use the generated constructors and the generated pass option struct. The hand written constructor got out of sync after some refactorings. Using a generated constructor and options struct ensures the everything remains in sync. Reviewed By: zero9178 This reverts commit 39da4682 and relands commit 771d9c05 which was originally reverted due to https://lab.llvm.org/buildbot#builders/61/builds/39694 Differential Revision: https://reviews.llvm.org/D143733
-
Aiden Grossman authored
-
Aiden Grossman authored
In the very initial version of the patch that introduced this test I had it only being conditionally compiled when in MLGO development mode (ie when HAVE_TFLITE is true), but after some revision it is enabled all the time and thus should be tested all the time. This patch fixes this leftover behavior that should've been fixed in the original change set.
-
Tobias Gysi authored
The commit breaks the mlir-vulkan runner: https://lab.llvm.org/buildbot#builders/61/builds/39694 This reverts commit 771d9c05.
-
Tobias Gysi authored
Update the FuncToLLVM pass to use the generated constructors and the generated pass option struct. The hand written constructor got out of sync after some refactorings. Using a generated constructor and options struct ensures the everything remains in sync. Reviewed By: zero9178 Differential Revision: https://reviews.llvm.org/D143733
-
Darshan Bhat authored
This change moves "DefaultVLIWScheduler" class declaration from DFAPacketizer.cpp to DFAPacketizer.h. This is needed because there is a protected class member of type "DefaultVLIWScheduler*" in "VLIWPacketizerList" class. The derived classes cannot use this memeber unless declaration is available to it. More specifically : // Without this change ``` class HexagonPacketizerList : public VLIWPacketizerList { public : HexagonPacketizerList() { // Below line will cause incomplete class error since // declaration was not available through header. VLIWScheduler->schedule(); } } ``` Reviewed By: kparzysz Differential Revision: https://reviews.llvm.org/D139767 -
Vlad Serebrennikov authored
Also mark [[ https://wg21.link/cwg2385 | CWG2385 ]] as `na`, because it eliminates wording inconsistency introduced by [[ https://wg21.link/cwg1111 | CWG1111 ]] resolution. Reviewed By: #clang-language-wg, erichkeane Differential Revision: https://reviews.llvm.org/D142315
-
Vlad Serebrennikov authored
-
Vlad Serebrennikov authored
[[ https://wg21.link/p1787 | P1787 ]]: "CWG2165 is resolved by removing the conflicting definition of it in [basic.scope]." Wording: [basic.namespace]/p1 changed and [basic.scope.declarative] removed entirely Reviewed By: #clang-language-wg, cor3ntin Differential Revision: https://reviews.llvm.org/D142717
-
Vlad Serebrennikov authored
This reverts commit d0e24f0c.
-
Vlad Serebrennikov authored
[[ https://wg21.link/p1787 | P1787 ]]: "CWG2165 is resolved by removing the conflicting definition of it in [basic.scope]." Wording: [basic.namespace]/p1 changed and [basic.scope.declarative] removed entirely Reviewed By: #clang-language-wg, cor3ntin Differential Revision: https://reviews.llvm.org/D142717
-
Tue Ly authored
Update macros/optimization.h build dependency for CMake and Bazel. Reviewed By: sivachandra Differential Revision: https://reviews.llvm.org/D143805
-
Tue Ly authored
-