- Jan 07, 2021
-
-
Andrzej Warzynski authored
As per [1]: ``` File names should use dashes, not underscores. ``` This patch updates the names of Flang driver tests accordingly. [1] https://github.com/llvm/llvm-project/blob/main/flang/docs/C%2B%2Bstyle.md
-
Caroline Concatto authored
This patch fixes a bug introduced in the patch: https://reviews.llvm.org/D93030 This patch pulls the test for scalable vector to be the first instruction to be checked. This avoids the Gather and Scatter cost model for AArch64 to compute the number of vector elements for something that is not a vector and therefore crashing.
-
Jan Svoboda authored
This is necessary for a future patch, where we start using this macro in another function. Reviewed By: dexonsmith Differential Revision: https://reviews.llvm.org/D93702
-
Alex Zinenko authored
-
Jan Svoboda authored
This patch moves `parseSimpleArgs` closer to `ParseDiagnosticArgs` so that sharing the parsing macro between them can be done more locally in a future patch. Reviewed By: dexonsmith Differential Revision: https://reviews.llvm.org/D94172
-
Jan Svoboda authored
Before this patch, ParseDiagnosticArgs can be called with a nullptr DiagnosticsEngine *. This happens early on in the compilation process, where no proper DiagnosticEngine exists, because the diagnostic options (passed through command line) are not known yet. This patch ensures nullptr is replaced by an ignoring DiagnosticEngine in ParseDiagnosticArgs, which allows to switch from pointer to a reference in some utility functions. Besides simplifying the code, this patch enables a future patch (D84673) that ports diagnostic options to the new marshalling infrastructure. Reviewed By: dexonsmith Differential Revision: https://reviews.llvm.org/D93701
-
Ivan Butygin authored
Add fastmath enum, attributes to some llvm dialect ops, `FastmathFlagsInterface` op interface, and `translateModuleToLLVMIR` support. Reviewed By: ftynse Differential Revision: https://reviews.llvm.org/D92485
-
Simon Pilgrim authored
-
Simon Pilgrim authored
Don't bother zeroing local (unused) variables just before returning. Fixes clang static analyzer warning.
-
Simon Pilgrim authored
-
Alexey Bataev authored
After merging the shuffles, we cannot rely on the previous shuffle anymore and need to shrink the final shuffle, if it is required. Reported in D92668 Differential Revision: https://reviews.llvm.org/D93967
-
Sam McCall authored
Currently some clients watch for CDB changes and restart clangd, now that we can reload compile_commands.json ourselves this is counterproductive. The capability allows this behavior to be phased out. This is going to be a mild regression, as we do not actually watch for files on disk and so new diagnostics need to wait until a rebuild is requested e.g. due to file change (and the internal caches have expired). However this is still a better tradeoff (and if it's important, we can request the client to watch files for us in the future). Differential Revision: https://reviews.llvm.org/D94222
-
Alex Zinenko authored
The original implementation of the OpenMP dialect to LLVM IR translation has been relying on a stack of insertion points for delayed insertion of branch instructions that correspond to terminator ops. This is an intrusive into ModuleTranslation and makes the translation non-local. A recent addition of the WsLoop translation exercised another approach where the parent op is responsible for converting terminators of all blocks in its regions. Use this approach for other OpenMP dialect operations with regions, remove the stack and deduplicate the code for converting such regions. Reviewed By: kiranchandramohan Differential Revision: https://reviews.llvm.org/D94086
-
Marek Kurdej authored
* The only exception is that the flag -std=c++2a is still used not to break compatibility with older compilers (clang <= 9, gcc <= 9). * Bump _LIBCPP_STD_VER for C++20 to 20 and use 21 for the future standard (C++2b). That's a preparation step to add c++2b support to libc++. Reviewed By: ldionne, #libc Differential Revision: https://reviews.llvm.org/D93383
-
Mikhail Maltsev authored
The nullptr_t_integral_cast.pass.cpp test is currently xfailed for C++03, but actually, it only fails with the first version of libc++ ABI. This patch changes XFAIL to UNSUPPORTED to avoid unexpected passes with ABI v2 or later. Reviewed By: ldionne, #libc Differential Revision: https://reviews.llvm.org/D93941
-
Simon Pilgrim authored
Attempt to simplify all/any-of style patterns that concatenate 2 smaller integers together into an and(x,y)/or(x,y) + icmp 0/-1 instead. This is mainly to help some bool predicate reduction patterns where we end up concatenating bool vectors that have been bitcasted to integers. Differential Revision: https://reviews.llvm.org/D93599
-
Jan Svoboda authored
Reviewed By: Bigcheese Differential Revision: https://reviews.llvm.org/D93700
-
Marek Kurdej authored
* Check created symlinks. Reviewed By: ldionne, #libc Differential Revision: https://reviews.llvm.org/D93520
-
Jan Svoboda authored
Reviewed By: dexonsmith Differential Revision: https://reviews.llvm.org/D93679
-
Andrzej Warzynski authored
This patch adds a frontend action for emitting object files. While Flang does not support code-generation, this action remains a placeholder. This patch simply provides glue-code to connect the compiler driver with the appropriate frontend action. The new action is triggered with the `-c` compiler driver flag, i.e. `flang-new -c`. This is then translated to `flang-new -fc1 -emit-obj`, so `-emit-obj` has to be marked as supported as well. As code-generation is not available yet, `flang-new -c` results in a driver error: ``` error: code-generation is not available yet ``` Hopefully this will help communicating the level of available functionality within Flang. The definition of `emit-obj` is updated so that it can be shared between Clang and Flang. As the original definition was enclosed within a Clang-specific TableGen `let` statement, it is extracted into a new `let` statement. That felt like the cleanest option. I also commented out `-triple` in Flang::ConstructJob and updated some comments there. This is similar to https://reviews.llvm.org/D93027. I wanted to make sure that it's clear that we can't support `-triple` until we have code-generation. However, once code-generation is available we _will need_ `-triple`. As this patch adds `-emit-obj`, the emit-obj.f90 becomes irrelevant and is deleted. Instead, phases.f90 is added to demonstrate that users can control compilation phases (indeed, `-c` is a phase control flag). Reviewed By: SouraVX, clementval Differential Revision: https://reviews.llvm.org/D93301
-
Jan Svoboda authored
Reviewed By: dexonsmith Differential Revision: https://reviews.llvm.org/D93698
-
David Spickett authored
Add optional memory tagging extension on AArch64. Use isAArch64() instead of listing the AArch64 triples, which fixes us not recognising aarch64_be. Reviewed By: omjavaid Differential Revision: https://reviews.llvm.org/D94084
-
Balázs Kéri authored
The assertion can happen if ASTImporter imports a CXXRecordDecl in a template and then imports another redeclaration of this declaration, while the first import is in progress. The process of first import did not set the "described template" yet and the second import finds the first declaration at setting the injected types. Setting the injected type requires in the assertion that the described template is set. The exact assertion was: clang/lib/AST/ASTContext.cpp:4411: clang::QualType clang::ASTContext::getInjectedClassNameType(clang::CXXRecordDecl*, clang::QualType) const: Assertion `NeedsInjectedClassNameType(Decl)' failed. Reviewed By: shafik Differential Revision: https://reviews.llvm.org/D94067
-
Jan Svoboda authored
Reapply "[clang][cli] Allow users to specify a conditional to prevent parsing options with MarshallingInfo" This reverts commit d0fa7a05 and fixes failing OptionMarshallingTest by adding the SHOULD_PARSE macro argument
-
Fraser Cormack authored
The patterns that want to use 'vnot' use a custom PatFrag. This is because 'vnot' uses immAllOnesV which implicitly uses BUILD_VECTOR rather than SPLAT_VECTOR. Reviewed By: craig.topper Differential Revision: https://reviews.llvm.org/D94078
-
Oliver Stannard authored
Reverting because this causes crashes on the 2-stage buildbots, for example http://lab.llvm.org:8011/#/builders/7/builds/1140. This reverts commit 9b228f10.
-
Jan Svoboda authored
Revert "[clang][cli] Allow users to specify a conditional to prevent parsing options with MarshallingInfo" This reverts commit 77db83ae
-
Jan Svoboda authored
This allows us to verify that we don't emit options multiple times. In most cases, that would be benign, but for options with `MarshallingInfoVectorString`, emitting wrong number of arguments might change the semantics. Reviewed By: Bigcheese Differential Revision: https://reviews.llvm.org/D93636
-
Jan Svoboda authored
Depends on D84189 & D93540. Reviewed By: Bigcheese Differential Revision: https://reviews.llvm.org/D84674
-
LLVM GN Syncbot authored
-
Artem Dergachev authored
This reverts commit 5663bf20. The cyclic dependency problem is addressed now. This is the ~fifth attempt to land this change.
-
Christian Sigg authored
The functions will be removed by January 20th. All call sites within MLIR have been converted in previous changes. Reviewed By: rriddle Differential Revision: https://reviews.llvm.org/D94191
-
Martin Storsjö authored
Differential Revision: https://reviews.llvm.org/D91138
-
Martin Storsjö authored
Differential Revision: https://reviews.llvm.org/D93950
-
Gil Rapaport authored
In response to https://reviews.llvm.org/D94088#inline-879268
-
River Riddle authored
This fixes document generation for type parameters.
-
Ganesh Gopalasubramanian authored
Differential Revision: https://reviews.llvm.org/D92812
-
Kazushi (Jam) Marukawa authored
-
Fangrui Song authored
-
Fangrui Song authored
-