- Mar 28, 2020
-
-
Richard Sandiford authored
Built-in SVE types are POD in much the same that scalars and fixed-length vectors are. Differential Revision: https://reviews.llvm.org/D76690
-
Simon Pilgrim authored
LowerSETCC handles strict cases as well, we don't have a separate function.
-
Artem Belevich authored
This reverts commit 6a9ad5f3. The patch breaks CUDA copmilation. Differential Revision: https://reviews.llvm.org/D76365
-
Alexandre Ganea authored
This fixes a number of warnings, where a function is re-defined after it is tagged as "being imported": D:\llvm-project\mlir\lib\ExecutionEngine\CRunnerUtils.cpp(24,17): warning: 'print_i32' redeclared without 'dllimport' attribute: 'dllexport' attribute added [-Winconsistent-dllimport] extern "C" void print_i32(int32_t i) { fprintf(stdout, "%" PRId32, i); } ^ D:\llvm-project\mlir\include\mlir/ExecutionEngine/CRunnerUtils.h(168,42): note: previous declaration is here extern "C" MLIR_CRUNNERUTILS_EXPORT void print_i32(int32_t i); ^ Differential Revision: https://reviews.llvm.org/D76654 -
LLVM GN Syncbot authored
-
jasonliu authored
Summary: Implement several XCOFF hooks to get '-r' option working for llvm-objdump -r. Reviewer: DiggerLin, hubert.reinterpretcast, jhenderson, MaskRay Differential Revision: https://reviews.llvm.org/D75131
-
Mikhail Maltsev authored
Summary: This patch: * adds tests for vreinterpret intinsics in big-endian mode * adds C++ runs to the CDE+MVE header compatibility test Reviewers: simon_tatham, MarkMurrayARM, ostannard, dmgreen Reviewed By: simon_tatham Subscribers: kristof.beyls, cfe-commits Tags: #clang Differential Revision: https://reviews.llvm.org/D76927
-
- Mar 27, 2020
-
-
Guillaume Chatelet authored
Summary: This is patch is part of a series to introduce an Alignment type. See this thread for context: http://lists.llvm.org/pipermail/llvm-dev/2019-July/133851.html See this patch for the introduction of the type: https://reviews.llvm.org/D64790 Reviewers: courbet Subscribers: arsenm, dschuff, sdardis, nemanjai, jvesely, nhaehnle, sbc100, jgravelle-google, hiraditya, aheejin, kbarton, jrtc27, atanasyan, jfb, kerbowa, llvm-commits Tags: #llvm Differential Revision: https://reviews.llvm.org/D76925
-
Walter Erquinigo authored
Summary: The DAP specifies the following for the SetBreakpoints request: The breakpoints returned are in the same order as the elements of the 'breakpoints' arguments This was not followed, as lldb-vscode was returning the breakpoints in a different order, because they were first stored into a map, and then traversed. Of course, maps normally don't preserve ordering. See this log I captured: --> {"command":"setBreakpoints", "arguments":{ "source":{ "name":"main.cpp", "path":"/Users/wallace/fbsource/xplat/sand/test-projects/buck-cpp/main.cpp" }, "lines":[6,10,11], "breakpoints":[{"line":6},{"line":10},{"line":11}], "sourceModified":false }, "type":"request", "seq":3 } <-- {"body":{ "breakpoints":[ {"id":1, "line":11,"source":{"name":"main.cpp","path":"xplat/sand/test-projects/buck-cpp/main.cpp"},"verified":true}, {"id":2,"line":6,"source":{"name":"main.cpp","path":"xplat/sand/test-projects/buck-cpp/main.cpp"},"verified":true}, {"id":3,"line":10,"source":{"name":"main.cpp","path":"xplat/sand/test-projects/buck-cpp/main.cpp"},"verified":true}]}, "command":"setBreakpoints", "request_seq":3, "seq":0, "success":true, "type":"response" } As you can see, the order was not respected. This was causing the IDE not to be able to disable/enable breakpoints by clicking on them in the breakpoint view in the lower corner of the Debug tab. This diff fixes the ordering problem. The traversal + querying was done very fast in O(nlogn) time. I'm keeping the same complexity. I also updated a couple of tests to account for the ordering. Reviewers: clayborg, aadsm, kusmour, labath Subscribers: lldb-commits Tags: #lldb Differential Revision: https://reviews.llvm.org/D76891 -
Louis Dionne authored
This one was left out from a previous commit.
-
Sam Parker authored
Given that some instructions generate wider result elements than their inputs, flag them as being able to generate non zeros in the false lanes. Differential Revision: https://reviews.llvm.org/D76766
-
Alexey Bataev authored
This reverts commit 36ed0cee to fix a crash in scan_messages.cpp test.
-
Alexandre Ganea authored
-
Louis Dionne authored
Since lit supports expanding substitutions recursively, we can define substitutions in terms of other substitutions. This allows us to simplify how libc++ substitutions are defined. This doesn't change the substitutions at all, it only makes them simpler to define.
-
Simon Pilgrim authored
-
Simon Pilgrim authored
-
Alexey Bataev authored
Added basic support (parsing/sema checks) for the inscan modifier in the reduction clauses.
-
Louis Dionne authored
lit is not very clever when it performs substitution on RUN lines. It simply looks for a match anywhere in the line (without tokenization) and replaces it by the expansion. This means that a RUN line containing e.g. `-verify-ignore-unexpected=note` wouod be expanded to `-verify-ignore-unexpected=<substitution for not>e`, which is surprising and nonsensical. It also means that something like `%compile_module` could be expanded to `<substitution-for-%compile>_module` or to the correct substitution, depending on the order in which substitutions are evaluated by lit. To avoid such problems, it is a good habit to delimit custom substitutions with some token. This commit does that for all substitutions used in the libc++ and libc++abi test suites.
-
Yannic Bonenberger authored
-
Alexandre Ganea authored
Before this patch, it wasn't possible to extend the ThinLTO threads to all SMT/CMT threads in the system. Only one thread per core was allowed, instructed by usage of llvm::heavyweight_hardware_concurrency() in the ThinLTO code. Any number passed to the LLD flag /opt:lldltojobs=..., or any other ThinLTO-specific flag, was previously interpreted in the context of llvm::heavyweight_hardware_concurrency(), which means SMT disabled. One can now say in LLD: /opt:lldltojobs=0 -- Use one std::thread / hardware core in the system (no SMT). Default value if flag not specified. /opt:lldltojobs=N -- Limit usage to N threads, regardless of usage of heavyweight_hardware_concurrency(). /opt:lldltojobs=all -- Use all hardware threads in the system. Equivalent to /opt:lldltojobs=$(nproc) on Linux and /opt:lldltojobs=%NUMBER_OF_PROCESSORS% on Windows. When an affinity mask is set for the process, threads will be created only for the cores selected by the mask. When N > number-of-hardware-threads-in-the-system, the threads in the thread pool will be dispatched equally on all CPU sockets (tested only on Windows). When N <= number-of-hardware-threads-on-a-CPU-socket, the threads will remain on the CPU socket where the process started (only on Windows). Differential Revision: https://reviews.llvm.org/D75153
-
Louis Dionne authored
-
Marcel Koester authored
[mlir] Extended Dominance analysis with a function to find the nearest common dominator of two given blocks. The Dominance analysis currently misses a utility function to find the nearest common dominator of two given blocks. This is required for a huge variety of different control-flow analyses and transformations. This commit adds this function and moves the getNode function from DominanceInfo to DominanceInfoBase, as it also works for post dominators. Differential Revision: https://reviews.llvm.org/D75507
-
Sam Parker authored
Add a flag for those instructions which read from the top/bottom halves of their inputs and produce a vector of results with double width elements. Differential Revision: https://reviews.llvm.org/D76762
-
Kirstóf Umann authored
Some checkers may not only depend on language options but also analyzer options. To make this possible this patch changes the parameter of the shouldRegister* function to CheckerManager to be able to query the analyzer options when deciding whether the checker should be registered. Differential Revision: https://reviews.llvm.org/D75271
-
Louis Dionne authored
This minor refactoring allows reducing the amount of processing that is duplicated when we re-run a flaky test. It also has the nice side effect that libc++'s current test format supports flaky .sh.cpp tests, because those are built on top of _runShTest, not executeShTest.
-
Louis Dionne authored
This allows defining substitutions in terms of other substitutions. For example, a %build substitution could be defined in terms of a %cxx substitution as '%cxx %s -o %t.exe' and the script would be properly expanded. Differential Revision: https://reviews.llvm.org/D76178
-
Sjoerd Meijer authored
This untangles the logic in widenIntOrFpInduction in order to make more explicit and visible how exactly the induction variable is lowered. Differential Revision: https://reviews.llvm.org/D76686
-
Guillaume Chatelet authored
Summary: This was discovered while converting to Align type. See this thread for context: http://lists.llvm.org/pipermail/llvm-dev/2019-July/133851.html See this patch for the introduction of the type: https://reviews.llvm.org/D64790 Reviewers: courbet Subscribers: hiraditya, llvm-commits Tags: #llvm Differential Revision: https://reviews.llvm.org/D76914
-
Kirstóf Umann authored
Exactly what it says on the tin! https://www.kernel.org/doc/htmldocs/kernel-api/API-kfree.html Differential Revision: https://reviews.llvm.org/D76917
-
Simon Pilgrim authored
Revert rGa3c715e9 "Twine - fix uninitialized variable warnings. NFCI." @dblaikie noticed that this may interfere with msan analysis
-
Simon Pilgrim authored
Revert rG6ff1ea32 "Fix "use of uninitialized variable" static analyzer warning. NFCI." @dblaikie noticed that this may interfere with msan analysis
-
Stephan Herhut authored
This change adds a new option to the StandardToLLVM lowering to configure the bitwidth of the index type independently of the target architecture's pointer size. Differential revision: https://reviews.llvm.org/D76353
-
Jonas Paulsson authored
-
David Green authored
This patterns seemed to be using the f32 instruction, not f16. Fix it to use the correct one. Differential Revision: https://reviews.llvm.org/D76841
-
Denis Khalikov authored
Summary: This patch adds support for 2D memref in mlir-vulkan-runner. Differential Revision: https://reviews.llvm.org/D76737
-
Georgii Rymar authored
We might have a crash scenario when we have an invalid DT_STRTAB value that is larger than the file size. I've added a test case to demonstrate. Differential revision: https://reviews.llvm.org/D76706
-
Alex Zinenko authored
Multiple operation conversions from the Standard dialect to the LLVM dialect are trivial one-to-one conversions that use only the pattern defined in base utility classes such as OneToOneConvertToLLVMPattern and VectorConvertToLLVMPattern. Use template aliases ("using" declarations) instead of creating derived classes without new functionality. -
Hans Wennborg authored
This fixes a regression from D69573 which broke the following example: $ echo 'operator C<T>*();' | bin/clang-format --style=Chromium operator C<T> *(); (There should be no space before the asterisk.) It seems the problem is in TokenAnnotator::spaceRequiredBetween(), which only looked at the token to the left of the * to see if it was a type or not. That code only handled simple types or identifiers, not templates or qualified types. This patch addresses that. Differential revision: https://reviews.llvm.org/D76850
-
Mikael Holmen authored
Summary: Unsigned types can alias the corresponding signed types. I don't see that this is explicitly mentioned in the Embedded-C specification, but I think it should work the same as for the integer types. Patch by: materi Reviewers: ebevhan, leonardchan Reviewed By: leonardchan Subscribers: kosarev, cfe-commits Tags: #clang Differential Revision: https://reviews.llvm.org/D76856
-
Guillaume Chatelet authored
Summary: This is patch is part of a series to introduce an Alignment type. See this thread for context: http://lists.llvm.org/pipermail/llvm-dev/2019-July/133851.html See this patch for the introduction of the type: https://reviews.llvm.org/D64790 Reviewers: courbet Reviewed By: courbet Subscribers: hiraditya, llvm-commits Tags: #llvm Differential Revision: https://reviews.llvm.org/D76625
-