- May 01, 2020
-
-
Julian Lettner authored
Properly print the test times histogram even for aborted (user interrupt, [Ctrl+C]) test runs.
-
Craig Topper authored
The vendor id is used to determine whether the processor supports hardware CRC32 in the Scudo code. The previous discussion about the patch is in [1], and more information about Hygon Dhyana processor is in[2]. [1]: https://reviews.llvm.org/D62368 [2]: https://git.kernel.org/torvalds/c/c9661c1e80b609cd038db7c908e061f0535804ef Patch by fanjinke (Jinke Fan) Differential Revision: https://reviews.llvm.org/D78874
-
Julian Lettner authored
Track and print the number of tests that were discovered but not executed due to test selection options: * --filter (regex filter) * --max-tests (limits number of tests) * sharding feature With this change all discovered tests are accounted for: every discovered test is included in one of the counts printed in the summary. Reviewed By: jdenny Differential Revision: https://reviews.llvm.org/D78078
-
Hubert Tong authored
The setting of `MCAsmInfo` properties for XCOFF got split between `MCAsmInfoXCOFF` and `PPCXCOFFMCAsmInfo`. Except for the properties that are dependent on the target information being passed via the constructor, the properties being set in `PPCXCOFFMCAsmInfo` had no fundamental reason for being treated as specific for XCOFF on PowerPC. Indeed, the property that might be considered more specific to PowerPC, `NeedsFunctionDescriptors`, was set in `MCAsmInfoXCOFF`. XCOFF being specific to PowerPC anyway, this patch consolidates the setting of the properties into `MCAsmInfoXCOFF` except for the cases that are dependent on the information provided via the `PPCXCOFFMCAsmInfo` constructor. This patch also reorders the assignments to the fields to match the declaration order in `MCAsmInfo`.
-
Hubert Tong authored
Summary: This patch splits mostly unrelated size constants into separate constexpr variables, sets explicit underlying types for the enumerations to match the fields they are used for, and improves various comments. This patch also replaces `<cname>` headers with `<name.h>` headers to match the usage of the declared names as global namespace members in the file. Reviewers: jasonliu, DiggerLin, daltenty, sfertile Reviewed By: jasonliu, sfertile Differential Revision: https://reviews.llvm.org/D79136
-
Craig Topper authored
This patch adds the x, t and g modifiers for inline asm from GCC. These will print a vector register as xmm*, ymm* or zmm* respectively. I also fixed register names with modifiers with inteldialect so they are no longer printed with a leading %. Patch by Amanieu d'Antras Differential Revision: https://reviews.llvm.org/D78977
-
Andrew Browne authored
-
Andrew Browne authored
This reverts commit ac2635e4. Revert was too hasty, build was already broken.
-
Craig Topper authored
Also fix some cost tables for vXi1 types to match the costs entries for the types they will be promoted to. Differential Revision: https://reviews.llvm.org/D79045
-
Andrew Browne authored
This reverts commit 25e2e922. Buildbot failure.
-
Andrew Browne authored
Summary: This was introduced in dda3c19a aka D77621. The unused template instantiation causes a warning on 32 bit systems about truncating a uint64_t to 32-bit size_t. Reviewed By: dblaikie, smeenai Differential Revision: https://reviews.llvm.org/D79214
-
Douglas Yung authored
Reviewers: craig.topper Reviewed By: craig.topper Subscribers: cfe-commits Tags: #clang Differential Revision: https://reviews.llvm.org/D79194
-
Shoaib Meenai authored
Android doesn't have a libgcc_s and uses libgcc instead, so adjust the build accordingly. This matches compiler-rt's build setup. libc++abi and libunwind were already checking for libgcc but in a different context. This change makes them search only for libgcc on Android now, but the code to link against libgcc if it were present was already there. Reviewed By: #libc, #libc_abi, #libunwind, rprichard, srhines Differential Revision: https://reviews.llvm.org/D78787
-
Diogo Sampaio authored
Summary: pr42199 When using regex_constants::match_prev_avail, it is defined that --first is valid, and match_not_bol and match_not_bow should be ignored. At the moment these flags are not ignored. This fixis that. Reviewers: ldionne, miyuki, EricWF, mclow.lists, #libc Reviewed By: ldionne, miyuki, #libc Subscribers: broadwaylamb, dexonsmith, libcxx-commits Tags: #libc Differential Revision: https://reviews.llvm.org/D75622
-
LLVM GN Syncbot authored
-
Wen-Heng (Jack) Chung authored
- Extract common logic between -convert-gpu-to-nvvm and -convert-gpu-to-rocdl. - Cope with the fact that alloca operates on different addrspaces between NVVM and ROCDL. - Modernize unit tests for ROCDL dialect. Differential Revision: https://reviews.llvm.org/D79021
-
Eric Astor authored
`rm` the file, THEN run the command that shouldn't create it, THEN confirm that it doesn't exist
-
Sergey Dmitriev authored
Summary: That makes AbstractCallSite::isCallee(const Use *) behavior consistent with AbstractCallSite constructor. Reviewers: jdoerfert Reviewed By: jdoerfert Subscribers: mgorny, llvm-commits Tags: #llvm Differential Revision: https://reviews.llvm.org/D79188
-
Nicolas Vasilache authored
Summary: This revision cleans up a layer of complexity in ScopedContext and uses InsertGuard instead of previously manual bookkeeping. The method `getBuilder` is renamed to `getBuilderRef` and spurious copies of OpBuilder are tracked. This results in some canonicalizations not happening anymore in the Linalg matmul to vector test. This test is retired because relying on DRRs for this has been shaky at best. The solution will be better support to write fused passes in C++ with more idiomatic pattern composition and application. Differential Revision: https://reviews.llvm.org/D79208
-
Evgenii Stepanov authored
Summary: Implement pattern initialization of memory (excluding the secondary allocator because it already has predictable memory contents). Expose both zero and pattern initialization through the C API. Reviewers: pcc, cryptoad Subscribers: #sanitizers, llvm-commits Tags: #sanitizers Differential Revision: https://reviews.llvm.org/D79133
-
aartbik authored
Summary: Added boolean support to vector.print. Useful for upcoming "mask" tests. Reviewers: ftynse, nicolasvasilache, andydavis1 Reviewed By: andydavis1 Subscribers: mehdi_amini, rriddle, jpienaar, shauheen, antiagainst, nicolasvasilache, arpith-jacob, mgester, lucyrfox, liufengdb, Joonsoo, grosul1, frgossen, Kayjukh, llvm-commits Tags: #llvm Differential Revision: https://reviews.llvm.org/D79198
-
Jonas Devlieghere authored
A patch command with the wrong -p value (strip prefix) created the nested lldb folder.
-
Nikita Popov authored
-
Arthur Eubanks authored
Summary: getCalledFunction() returns null for indirect function invocations. Subscribers: hiraditya, llvm-commits Tags: #llvm Differential Revision: https://reviews.llvm.org/D79203
-
Michael Liao authored
-
Sam Clegg authored
Reverting part of https://reviews.llvm.org/D79137 which caused an failure in an ObjectYAML test.
-
Florian Hahn authored
This cleanup was suggested as part of D78458.
-
Nikita Popov authored
In D74183 clang started emitting alignment for sret parameters unconditionally. This caused a 1.5% compile-time regression on tramp3d-v4. The reason is that we now generate many instance of IR like %ptrint = ptrtoint %class.GuardLayers* %guards_m to i64 %maskedptr = and i64 %ptrint, 3 %maskcond = icmp eq i64 %maskedptr, 0 tail call void @llvm.assume(i1 %maskcond) to preserve the alignment information during inlining. Based on IR analysis, these assumptions also regress optimization. The attached phase ordering test case illustrates two issues: One are instruction count based optimization heuristics, which are affected by the four additional instructions of the assumption. The other is blocking of SROA due to ptrtoint casts (PR45763). We already encountered the same problem in Rust, where we (unlike Clang) generally prefer to emit alignment information absolutely everywhere it is available. We were only able to do this after hardcoding -preserve-alignment-assumptions-during-inlining=false, because we were seeing significant optimization and compile-time regressions otherwise. This patch disables -preserve-alignment-assumptions-during-inlining by default, because we should not be punishing people for adding more alignment annotations. Once the assume bundle work shakes out and we can represent (and use) alignment assumptions using assume bundles, it should be possible to re-enable this with reduced overhead. Differential Revision: https://reviews.llvm.org/D76886 -
Eli Friedman authored
-
Julian Lettner authored
This also avoids assigning to the option object.
-
mydeveloperday authored
[clang-format] [PR45626] SpacesInAngles does not insert or preserve leading space before :: operator Summary: See https://bugs.llvm.org/show_bug.cgi?id=45626 Ensure space between < and :: (void)static_cast<::std::uint32_t >(1); Reviewers: krasimir, mitchell-stellar Reviewed By: krasimir Subscribers: cfe-commits Tags: #clang, #clang-format Differential Revision: https://reviews.llvm.org/D79172
-
Jonas Devlieghere authored
Currently, `SBCommandInterpreterRunOptions` is defined in `SBCommandInterpreter.h`. Given that the options are always passed by reference, a forward declaration is sufficient. That's not the case for `SBCommandInterpreterRunResults`, which we need for a new overload for `RunCommandInterpreter` and that returns this new class by value. We can't include `SBCommandInterpreter.h` because `SBCommandInterpreter::GetDebugger()` returns SBDebugger by value and therefore needs a full definition. This patch moves the definition of `SBCommandInterpreterRunOptions` into a new header. In a later patch, `SBCommandInterpreterRunResults` will be defined in there as well, solving the aforementioned problem. Differential revision: https://reviews.llvm.org/D79115
-
Alexey Bataev authored
Summary: Added parsing/sema/serialization supoprt for uses_allocators clause. Reviewers: jdoerfert Subscribers: yaxunl, guansong, arphaman, cfe-commits, caomhin Tags: #clang Differential Revision: https://reviews.llvm.org/D78577
-
Jonas Devlieghere authored
This implements Greg's suggestion from D78825 to include "auto handle events" and "spawn thread" in CommandInterpreterRunOptions. This change is in preparation for adding a new overload for RunCommandInterpreter that takes only SBCommandInterpreterRunOptions and returns SBCommandInterpreterRunResults. Differential revision: https://reviews.llvm.org/D79108
-
Jonas Devlieghere authored
It seems like only the unittests are building with BUILD_WITH_INSTALL_RPATH set to OFF. Of course when I did my last change I only ran check-lldb-unit. Not sure why this difference exists, why would you even install the unittest? For the LLDB framework we do need different build and install RPATHs. Currently that logic lives downstream. I plan to upstream that in the near future. For now I'm just trying to make it possible to run the test.
-
Sylvain Audi authored
[clang] Disable check for system sanitizer blacklists files if -fno-sanitizer-blacklist was specified This is to avoid checking for the validity of a file that is not used. This also contains a minor fix for the test, as the cfi sanitizer requires -flto and -fvisibility= arguments. Differential Revision: https://reviews.llvm.org/D79043
-
Louis Dionne authored
The libcxx.util utilities don't work properly, and we should remove them when we get rid of compiler.py. In particular, libcxx.util.to_string appears to be completely broken.
-
Masoud Ataei authored
Fixing test suite of the committed PR: https://reviews.llvm.org/D78054. I am proposing to remove the PowerPC target triple in the test suite. Reviewed by: @jsji, @fpetrogalli Tags: LLVM Differential Revision: https://reviews.llvm.org/D79124
-
Sam Clegg authored
This change add support for defined wasm globals in the .s format, the MC layer, and wasm-ld Currently there is no support custom initialization and all wasm globals are initialized to zero. Fixes: PR45742 Differential Revision: https://reviews.llvm.org/D79137
-
Louis Dionne authored
19123a3e was too naive -- we really want to handle both Python 2 and Python 3, not only Python 3.
-