- Jun 20, 2020
-
-
Xing GUO authored
This patch helps add support for emitting the .debug_info section to yaml2elf. Reviewed By: jhenderson, grimar, MaskRay Differential Revision: https://reviews.llvm.org/D82073
-
Carl Ritson authored
Always prefer to clobber input SGPRs and restore them after the spill. This applies to both spills to VGPRs and scratch. Reviewed By: arsenm Differential Revision: https://reviews.llvm.org/D81914
-
Thomas Raoux authored
-
Weverything authored
-
peter klausler authored
Fix F output editing in the Fortran runtime so that it respects the 'd' count of digits after the decimal mark. Reviewed By: tskeith Differential Revision: https://reviews.llvm.org/D82242
-
Thomas Raoux authored
Subview operations are not natively supported downstream in the spirv path. This change allows removing subview when used by vector transfer the same way we already do it when they are used by LoadOp/StoreOp Differential Revision: https://reviews.llvm.org/D82106
-
Thomas Raoux authored
Allow lhs and rhs to have different type than accumulator/destination. Some hardware like GPUs support natively operations like uint8xuint8xuint32. Differential Revision: https://reviews.llvm.org/D82069
-
Benson Chu authored
Clang is missing one of the conditions for C99 6.5.9p2, where comparison between pointers must either both point to incomplete types or both point to complete types. This patch adds an extra check to the clause where two pointers are of compatible types. This only applies to C89/C99; the relevant part of the standard was rewritten for C11. Differential Revision: https://reviews.llvm.org/D79945
-
Ryan Prichard authored
At this point in this code: - COMPILER_RT_DEFAULT_TARGET_TRIPLE is "i686-linux-android" - arch is "i386" The get_compiler_rt_target function currently turns that into: i686-android-linux-android The ${COMPILER_RT_OS_SUFFIX} is "-android" and redundant, so stop adding it. The get_compiler_rt_target() function is used for the LLVM_ENABLE_PER_TARGET_RUNTIME_DIR mode that isn't normally used with Android. Differential Revision: https://reviews.llvm.org/D82150 -
Ryan Prichard authored
Differential Revision: https://reviews.llvm.org/D82146
-
romanova-ekaterina authored
This is a fix for PR #46392 (Diagnostic message (error) related to ThinLTO caching needs to be downgraded to a remark). There are diagnostic messages related to ThinLTO caching that contain the word "error", but they are really just notices/remarks for users, and they don't cause a build failure. The word "error" appearing can be confusing to users, and may even cause deeper problems. User's build system might be designed to interpret any error messages (even a benign error message as the one above) reported by the compiler as a build failure, thus causing the build to fail "needlessly". In short, the term "error" in this diagnostic is misleading at best, and may be causing build systems to fail at worst. Differential Revision: https://reviews.llvm.org/D82138
-
Eric Christopher authored
migrate away from the use of blacklist and whitelist.
-
Eric Christopher authored
migrate away from the use of blacklist and whitelist.
-
Eric Christopher authored
migrate away from the use of blacklist and whitelist.
-
Eric Christopher authored
migrate away from the use of blacklist and whitelist.
-
Heejin Ahn authored
When created in RegStackify pass, `TEE` has two destinations, where op0 is stackified and op1 is not. But it is possible that op0 becomes unstackified in `fixUnwindMismatches` function in CFGStackify pass when a nested try-catch-end is introduced, violating the invariant of `TEE`s destinations. In this case we convert the `TEE` into two `COPY`s, which will eventually be resolved in ExplicitLocals. Reviewed By: dschuff Differential Revision: https://reviews.llvm.org/D81851
-
Eric Christopher authored
migrate away from the use of blacklist and whitelist.
-
Eric Christopher authored
migrate away from the use of blacklist and whitelist.
-
Eric Christopher authored
migrate away from the use of blacklist and whitelist.
-
Martin Storsjö authored
This fixes cross building on a case sensitive file system after 2e613d2d. (The official Windows SDKs don't have self-consistent casing and can't be used as such on case sentisive file systems without case fixups, while mingw headers consistently use lower case.)
-
Rahul Joshi authored
Differential Revision: https://reviews.llvm.org/D82229
-
Arthur Eubanks authored
Summary: Looks like these were for the most part touched over a decade ago. Subscribers: mgorny, cfe-commits, llvm-commits Tags: #clang, #llvm Differential Revision: https://reviews.llvm.org/D82231
-
Raphael Isemann authored
-
peter klausler authored
Implement rest of DATA statement semantics and conversion of DATA statement initializations into static initializers of objects in their symbol table entries. Reviewed By: tskeith, PeteSteinfeld Differential Revision: https://reviews.llvm.org/D82207
-
Amara Emerson authored
We were defaulting to the lower action for this, resulting in SHL+ASHR sequences. On AArch64 we can do this in one instruction for an arbitrary extension using SBFM as we do for G_SEXT. Differential Revision: https://reviews.llvm.org/D81992
-
cgyurgyik authored
Reviewed-by: sivachandra Differential Revision: https://reviews.llvm.org/D82134
-
Daniel authored
when both a check and its alias are enabled, we should only take the fixes of one of them and not both. This patch fixes bug 45577 https://bugs.llvm.org/show_bug.cgi?id=45577 Reviewed By: aaron.ballman, njames93 Differential Revision: https://reviews.llvm.org/D80753
-
Sanjay Patel authored
-
Yitzhak Mandelbaum authored
Summary: Currently, `cat` validates range selections before extracting the corresponding source text. However, this means that any range inside a macro is rejected as an error. This patch changes the implementation to first try to map the range to something reasonable. This makes the behavior consistent with handling of ranges used for selecting portions of the source to edit. Also updates a clang-tidy lit-test for one of the checks which was affected by this change. Reviewers: gribozavr2, tdl-g Subscribers: cfe-commits Tags: #clang Differential Revision: https://reviews.llvm.org/D82126
-
Eric Schweitz authored
The symbol map is a data structure for tracking variables in a subprogram during the lowering of that subprogram to FIR/MLIR. These data structures will be used by the bridge, which has not been upstreamed yet. Differential revision: https://reviews.llvm.org/D82140
-
Jonas Devlieghere authored
-
Thomas Preud'homme authored
This was originally cd2553de and reverted in a3adfb40. The ADT itostr bug this triggered was fixed in f3e8f961.
-
aartbik authored
Use direct vector constants for the 1-D case. This approach scales much better than generating elaborate insertion operations that are eventually folded into a constant. We could of course generalize the 1-D case to higher ranks, but this simplification already helps in scaling some microbenchmarks that would formerly crash on the intermediate IR length. Reviewed By: reidtatge Differential Revision: https://reviews.llvm.org/D82144
-
Thomas Preud'homme authored
Summary: UBSan buildbot caught an undefined behavior in itostr with INT64_MIN. The negation cannot be represented in the promoted operand (long long). Negation is well defined on unsigned value though so this commit does the negation after the static cast. Reviewers: jhenderson, chandlerc, lattner Reviewed By: lattner Subscribers: dexonsmith, llvm-commits Tags: #llvm Differential Revision: https://reviews.llvm.org/D82200
-
Raphael Isemann authored
Summary: LLVM is using its own isPrint/isSpace implementation that doesn't change depending on the current locale. LLDB should do the same to prevent that internal logic changes depending on the set locale. Reviewers: JDevlieghere, labath, mib, totally_not_teemperor Reviewed By: JDevlieghere Differential Revision: https://reviews.llvm.org/D82175
-
Raphael Isemann authored
Summary: Since commit 7b3ef05a the Objective-C++ plugin is dead code. That commit added Objective-C++ to the list of languages for which `Language::LanguageIsCPlusPlus` returns true. As the C++ language plugin also uses that method to figure out if it is responsible for a given language, the C++ plugin since then also became the plugin that we found when looking for a language plugin for Objective-C++. The only real fallout from that is that the source highlighting for Objective-C++ files never worked as we always found the C++ plugin which refuses to highlight files with Objective-C++ extensions. This patch just adds a special exception for Objective-C++ to the list of languages that are governed by the C++ plugin. Also adds a test that makes sure that we find the right plugin for all C language types and that the highlighting for `.mm` (Objective-C++) and `.m` (Objective-C) files works. I didn't revert 7b3ef05a as it does make sense to return true for Objective-C++ from `Language::LanguageIsCPlusPlus` (e.g., we currently check if we care about ODR violations by doing `if (Language::LanguageIsCPlusPlus(...))` and this should also work for Objective-C++). Fixes rdar://64420183 Reviewers: aprantl Reviewed By: aprantl Subscribers: mgorny, abidh, JDevlieghere Differential Revision: https://reviews.llvm.org/D82109
-
Raphael Isemann authored
Summary: TerminalSizeChanged is called from our SIGWINCH signal handler but the IOHandlerEditline currently doesn't check if we are actually using the real editline backend. If we're not using the real editline backend, `m_editline_up` won't be set and `IOHandlerEditline::TerminalSizeChanged` will access the empty unique_ptr. In a real use case we don't use the editline backend when we for example read input from a file. We also create some temporary IOHandlerEditline's during LLDB startup it seems that are also treated as non-interactive (apparently to read startup commands). This patch just adds a nullptr check for`m_editline_up` as we do in the rest of IOHandlerEditline. Fixes rdar://problem/63921950 Reviewers: labath, friss Reviewed By: friss Subscribers: abidh, JDevlieghere Differential Revision: https://reviews.llvm.org/D81729
-
Lang Hames authored
This relaxes an assertion that required symbols to start before the end of a block. Instead, symbols are now required to end on or before the end of a block. This fixes two important corner cases: Symbols at the start of empty blocks/sections, and block/section end symbols.
-
Lang Hames authored
This can be helpful for sanity checking JITLink memory manager behavior.
-
Adrian Prantl authored
and delete a bunch (but not all) redundant code. If you compare the remaining implementations of Platform*Simulator.cpp, there is still an obvious leftover cleanup task. Specifically, this patch - removes SDK initialization from dotest (there is equivalent but more complete code in Makefile.rules) - make Platform*Simulator inherit the generic implementation of PlatformAppleSimulator (more can be done here) - simplify the platform logic in Makefile.rules - replace the custom SDK finding logic in Platform*Simulator with XcodeSDK - adds a test for each supported simulator Differential Revision: https://reviews.llvm.org/D81980
-