- Jul 04, 2023
-
-
Peter Klausler authored
OPEN statements can be used to change some, but not all, attributes of units that have already been opened. The I/O runtime library wasn't allowing ENCODING= to be changed. Every other Fortran compiler permits this usage, and it's safe and useful, so allow it. (Otherwise there's no good way to ensure that the preconnected unit 6 is in UTF-8 mode.) Differential Revision: https://reviews.llvm.org/D154379
-
Peter Klausler authored
We intentionally process NAMELIST groups in a scope after having resolved all of the names in that scope. This means that a name whose first appearance in a scope is in the NAMELIST group resolves to a local object, if any, rather than to any host associated object. The standard is unclear on this point, and there is no clear precedent in other compilers. This patch doesn't implement this choice -- that was done long ago -- but just documents the behavior in Extensions.md. Differential Revision: https://reviews.llvm.org/D154375
-
Peter Klausler authored
The code that folds the intrinsic function HYPOT was neglecting to warn the programmer about overflow when it occurs. Differential Revision: https://reviews.llvm.org/D154371
-
Alex Langford authored
We already have LLDB_DEPRECATED which is used to mark methods as deprecated with a message and an alternative to use instead. This is expresses an intent of "We recognize this functionality is useful but there are some pitfalls with the interface we have exposed." In other cases, there are no "alternative" methods to use and the code should be refactored to avoid using a method entirely. For example, `SBValue::Cast` should be avoided in favor of using the expression evaluator to perform a cast. There isn't a mechanical solution, the recommendation is to instead refactor your code. This commit renames the existing `LLDB_DEPRECATED` to `LLDB_DEPRECATED_FIXME`, and adds a `LLDB_DEPRECATED` macro to cover the second scenario. Differential Revision: https://reviews.llvm.org/D153928
-
Alex Langford authored
I'm not convinced that it makes sense for the paths to be ConstStrings. We're going to be putting them into FileSpecs (which are backed by ConstStrings, for now) but otherwise there's no need to store them as ConstStrings upfront. Differential Revision: https://reviews.llvm.org/D153827
-
Florian Hahn authored
Based on @peixin test case shared in D153004.
-
Peter Klausler authored
Add a document that summarizes Fortran 202X's upcoming features and their urgency for implementation. Differential Revision: https://reviews.llvm.org/D153916
-
Johannes Doerfert authored
Before we ended up not calling the specialized AAType::isImpliedByIR but the generic IRAttribute::isImpliedByIR in the IRAttribute::initialize function.
-
Johannes Doerfert authored
-
Johannes Doerfert authored
-
Johannes Doerfert authored
In a follow up we will reuse the logic in MemoryEffectsBase to merge AAMemoryLocation and AAMemoryBehavior without duplicating all the bit fiddling code already available in MemoryEffectsBase. Reviewed By: nikic Differential Revision: https://reviews.llvm.org/D153305
-
Alex Langford authored
Instead of just returning a raw `const char *`, I think llvm::StringRef would make more sense. Most of the time that we use the return value of `GetProcessPluginName` we're passing it to `CreateProcess` which takes a StringRef anyway. Differential Revision: https://reviews.llvm.org/D153825
-
Slava Zakharin authored
Actual argument expressions must be evaluated before the invocation of the sequence of per-element calls of an impure elemental subprogram. Otherwise, the side effects of the calls may affect the input for the consequent elements. The proposed changes are described by Jean in D154174. Reviewed By: tblah Differential Revision: https://reviews.llvm.org/D154263
-
Kiran Chandramohan authored
-
Florian Hahn authored
Update trip count of test in pr56319-vector-exit-cond-optimization-epilogue-vectorization.ll to make sure epilogue vectorization will still trigger after D154261, checking for the original issue. Move the original test to limit-vf-by-tripcount.ll for testing new functionality of D154261.
-
Dmitriy Smirnov authored
This patch lowers allocatables and pointers named in "private" OpenMP clause. Reviewed By: kiranchandramohan Differential Revision: https://reviews.llvm.org/D148570
-
Jorge Pinto Sousa authored
Index accessing checks are not performed for aliases of `std::array`, as only `std::array` itself seems to be checked. This patch aims to extend it for aliases such as: `using MyArray = std::array<int, 10>;` Reviewed By: PiotrZSL Differential Revision: https://reviews.llvm.org/D154297
-
Mike Crowe authored
Victor Zverovich pointed out[1] that printf takes the field width and precision arguments before the value to be printed whereas std::print takes the value first (unless positional arguments are used.) Many of the test cases in use-std-print.cpp were incorrect. Teach the check to rotate the arguments when required to correct this. Correct the test cases and add more. [1] https://github.com/fmtlib/fmt/pull/3515#issuecomment-1615259893 Reviewed By: PiotrZSL Differential Revision: https://reviews.llvm.org/D154283
-
Alex Langford authored
This doesn't really use fast comparison or string uniqueness. In fact, all of the current callers pass an empty string for type_name. The only reason I don't remove it is because it looks like it is used downstream for swift. Differential Revision: https://reviews.llvm.org/D153810
-
Alex Langford authored
2/3 of the ConstStrings in this class were just to be able to log something. Putting something in the StringPool just to log it doesn't make a lot of sense, so let's remove them. The remaining use is for `RegisterPersistentDecl` which is fine for now. Differential Revision: https://reviews.llvm.org/D153905
-
Peter Klausler authored
The soon-to-be-published next revision of the ISO Fortran language standard contains a couple of breaking changes to previous specifications that may cause existing programs to silently change their behavior. For the change that introduces automatic reallocation of deferred length allocatable character scalar variables when they appear as the targets of internal WRITE statements, as IOMSG=/ERRMSG= variables, as outputs of INQUIRE specifiers, or as INTENT(OUT) arguments to intrinsic procedures, this patch adds an optional portability warning. Differential Revision: https://reviews.llvm.org/D154242
-
Matthias Springer authored
There are existing implementations for `scf.for`, `scf.forall` and `affine.for`. This revision adds an interface method to the `LoopLikeOpInterface`. * `scf.forall` now implements the `LoopLikeOpInterface`. * The implementations of `scf.for` and `scf.forall` become interface method implementations. `affine.for` remains as is for the moment. (The implementation of `promoteIfSingleIteration` depepends on helper functions from `MLIRAffineAnalysis`, which cannot be used from `MLIRAffineDialect`, where the interface is currently implemented.) * More efficient implementations of `promoteIfSingleIteration`. In particular, the `scf.forall` operation now inlines operations instead of cloning them. This also preserves handles when used from the transform dialect. Differential Revision: https://reviews.llvm.org/D154343
-
- Jul 03, 2023
-
-
Christudasan Devadasan authored
It is needed for target specific initializatons. Reviewed By: qcolombet Differential Revision: https://reviews.llvm.org/D143758
-
Yashwant Singh authored
This is a prep patch for D150388. Treating rddsp and wrdsp as copy instructions was causing test failures as we tried using isCopyInstr() hook to query target-specific copy instructions for LiveRangeSplitting. As suggested, removing 'isMoveReg = 1' from wrdsp and rddsp so they aren't considered simple copy-like instructions for the moment. Reviewed By: sdardis Differential Revision: https://reviews.llvm.org/D151181
-
Christian Ulmann authored
This commit ensures that the garbage collector attribute of LLVMFuncOps is exported, even when they are declarations.
-
Kadir Cetinkaya authored
This also does some cleanups, I am happy to undo them (or send as separate patches): - Change the early exit to stop only once we hit an expansion inside the main file, to make sure we keep following the nested expansions. - Add more tests to cover all the cases mentioned in the implementation - Drop the adjustments for prev/next tokens. We do the final checks based on the expansion locations anyway, so any intermediate mapping was a no-op. Differential Revision: https://reviews.llvm.org/D154335
-
David Truby authored
This patch simply adds a cmake install line for omp_lib.h that was previously missing, to put it alongisde omp_lib.mod so it can be found by the driver.
-
Paul Walker authored
-
Joel E. Denny authored
In the case of partially mapped structs, libomptarget sometimes adds padding to device allocations to ensure they are aligned properly. However, without this patch, it considers that padding to be mapped to the host, which can cause presence checks (e.g., `omp_target_is_present` or a `present` modifier) to misbehave for unmapped parts of the struct. This patch keeps the padding but treats it as unmapped. See the new test case for examples. Reviewed By: grokos, jdoerfert Differential Revision: https://reviews.llvm.org/D149685
-
Mariya Podchishchaeva authored
Add `__has_extension (cxx_defaulted_functions)` and `__has_extension (cxx_default_function_template_args)` since they are accepted in C++98 mode as extensions. Fixes https://github.com/llvm/llvm-project/issues/61758 Reviewed By: aaron.ballman Differential Revision: https://reviews.llvm.org/D154334
-
Balazs Benics authored
Differential Revision: https://reviews.llvm.org/D152891
-
David Spickett authored
This reverts commit 92a9c30c. This has caused a test failure in the 2nd stage of Linaro's Arm 32 bit buildbots. LLVM::simplified-template-names.s 7: error: Simplified template DW_AT_name could not be reconstituted: check:10'0 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ 8: original: f3<unsigned char, (unsigned char)'\x00'> check:10'0 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ 9: reconstituted: f3<unsigned char, (unsigned char)'\x7f'> check:10'0 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ I suspect a load/store is slightly off.
-
Louis Dionne authored
If someone encounters issues due to this patch (e.g. they are using that test format out-of-tree), please reach out on the review and we'll figure out a way forward. Differential Revision: https://reviews.llvm.org/D153534
-
Louis Dionne authored
I made sure they all had some expected-error output in them. Many of these tests would be better implemented as a positive test using SFINAE, but this is beyond the scope of this patch. Differential Revision: https://reviews.llvm.org/D153980
-
Louis Dionne authored
Those were found while trying to enable configurations like no-threads and no-localization with Clang modules enabled. Differential Revision: https://reviews.llvm.org/D153977
-
Matthias Springer authored
This op applies loop-invariant code motion to the targeted loop-like op. Differential Revision: https://reviews.llvm.org/D154327
-
Florian Hahn authored
Update the test to check for the vectorization call in the main vector loop, not the dead epilogue vector loop as it does currently.
-
Simon Pilgrim authored
Similar to the existing _mm_load1_pd/_mm_loaddup_pd and broadcast vector loads, these intrinsic should ensure the loads are unaligned and not assume type alignment Fixes #62325
-
Felipe de Azevedo Piovezan authored
A very old commit (9422dd64) changed the signature of this function in a number of ways. This patch aims to improve it: 1. Reword the documentation, which still mentions old parameters that no longer exist, and to elaborate upon the behavior of this function. 2. Remove the unnecessary parameter `op_addr_idx`. This parameter is odd in a couple of ways: we never use it with a value that is non-zero, and the matching `Update_DW_OP_addr` function doesn't use a similar parameter. We also document that this new behavior. If we ever decide to handle multiple "DW_OP_addr", we can introduce the complexity again. Differential Revision: https://reviews.llvm.org/D154265
-
Felipe de Azevedo Piovezan authored
D68678 added a test that ensures an Apple accelerator lookup is done efficiently. Since these tables are not used for DWARF 5, we should decorate the test appropriately. Differential Revision: https://reviews.llvm.org/D154268
-