- Jan 07, 2022
-
-
Dave Lee authored
Fixes incomplete command names in `apropos` results. The full command names given by `apropos` have come from command name string literals given to `CommandObject` constructors. For most commands, this has been accurate, but some commands have incorrect strings. This results in `apropos` output that doesn't tell the user the full command name they might want learn more about. These strings can be fixed. There's a seperate issue that can't be fixed as easily: plugin commands. With the way they're implemented, plugin commands have to exclude the root command from their command name string. To illustrate, the `language objc` subcommand has to set its command name string to "objc", which results in apropos printing results as `objc ...` instead of `language objc ...`. To fix both of these issues, this commit changes `FindCommandsForApropos` to derive the fully qualified command name using the keys of subcommand maps. Differential Revision: https://reviews.llvm.org/D116491
-
Shao-Ce SUN authored
Reviewed By: craig.topper Differential Revision: https://reviews.llvm.org/D116719
-
Philip Reames authored
This reverts commit 9ce30fe8. Appears to be causing a problem on a buildbot, revert while investigating. https://green.lab.llvm.org/green//job/clang-stage1-RA/26818/consoleFull#-1502953973d489585b-5106-414a-ac11-3ff90657619c
-
Jim Lin authored
-
Kazu Hirata authored
This patch fixes: mlir/include/mlir/Dialect/Linalg/Transforms/Transforms.h:913:30: error: private field 'options' is not used [-Werror,-Wunused-private-field]
-
Yusra Syeda authored
Differential Revision: https://reviews.llvm.org/D115269
-
Liqin Weng authored
Reviewed By: craig.topper Differential Revision: https://reviews.llvm.org/D115133
-
Philip Reames authored
This is a reoccuring pattern, we can consolidate three copies into one. The main motivation is to reduce usages of isMallocLike.
-
Philip Reames authored
This parameter took the non-default value exactly twice, and neither had semantic effect.
-
Philip Reames authored
These are implementation details of the global-opt transform and not easily reuseable, so remove them from the analysis header.
-
Philip Reames authored
-
Philip Reames authored
-
Philip Reames authored
-
Nicolas Vasilache authored
-
Nikolas Klauser authored
Reviewed By: Quuxplusone, #libc, Mordante Spies: mzeren-vmw, ckennelly, arichardson, ldionne, Mordante, libcxx-commits, Quuxplusone Differential Revision: https://reviews.llvm.org/D113013
-
Andrew Browne authored
Reviewed By: morehouse Differential Revision: https://reviews.llvm.org/D116761
-
Nicolas Vasilache authored
Differential Revision: https://reviews.llvm.org/D116763
-
-
Arlo Siemsen authored
Microsoft has added several new entries to the CV_CFL_LANG enum, including Rust: https://docs.microsoft.com/en-us/visualstudio/debugger/debug-interface-access/cv-cfl-lang This change adds Rust to the corresponding LLVM enum and translates `dwarf::DW_LANG_Rust` to `SourceLanguage::Rust` in the CodeView AsmPrinter. This means that Rust will no longer emit as Masm. Differential Revision: https://reviews.llvm.org/D115300 -
Colin LeMahieu authored
The lld testcase change from ddf1fb1f should take care of the build breakage from before.
-
Brian Cain authored
Previously compounding was all-or-nothing. Now, the compounding attempts will iterate and yield the most compounds that still result in a valid packet.
-
Matthias Springer authored
This has two advantages. 1. It is more efficient. No need to clone the entire region. 2. Recreating ops (via cloning) invalidates analysis results. Previously, an OpResult could have bufferized out-of-place, even though the analysis requested an in-place bufferization. That is because BufferizationState keeps track of OpResults for storing bufferization analysis results (and cloned ops have new OpResults). Differential Revision: https://reviews.llvm.org/D116453
-
Alexandre Ganea authored
D:\git\llvm-project\clang\unittests\Analysis\FlowSensitive\MultiVarConstantPropagationTest.cpp(104) : warning C4715: 'clang::dataflow::`anonymous namespace'::operator<<': not all control paths return a value
-
Matthias Springer authored
In addition, all functions that call `allocationFn` now return FailureOr<Value>. This resolves a few TODOs in the code base. Differential Revision: https://reviews.llvm.org/D116452
-
Nicolas Vasilache authored
Tiling patterns can be reduced to a single pattern by using interface-based patterns. Differential Revision: https://reviews.llvm.org/D116733
-
Matthias Springer authored
Differential Revision: https://reviews.llvm.org/D116451
-
Christian Sigg authored
Update prettyprinters.py to match MLIR changes. This has gone unnoticed because no build bot is running tests with debug info. I will look into what we can do about this separately. There is https://green.lab.llvm.org/green/view/LLDB/job/lldb-cmake/, from Apple. The Debug Info tests are failing despite the green result. See https://github.com/llvm/llvm-project/issues/48872. Note: the llvm-support.gdb test only works with Debug, but not RelWithDebInfo because some checked symbols are stripped. Reviewed By: dblaikie Differential Revision: https://reviews.llvm.org/D116646
-
Matthias Springer authored
If `createDealloc` is deactivated (enabled by default), newly allocated buffers are not deallocated anymore. In such a case, the missing deallocations can be inserted by the existing "BufferDeallocation" pass. This change is needed for unifying core bufferization and Comprehensive Bufferize. Core bufferization has a separate pass for generating deallocations. Note: In the future, this will evolve towards generating deallocation ops only for buffer allocations that do not escape block boundaries (i.e., that are in destination passing style). Differential Revision: https://reviews.llvm.org/D116450
-
Michael Spencer authored
This re-lands: - 04192422 - 015e08c6 Which I reverted in ea835171 in error. Differential Revision: https://reviews.llvm.org/D114206
-
Congzhe Cao authored
There was a limitation in legality that in the original inner loop latch, no instruction was allowed between the induction variable increment and the branch instruction. This is because we used to split the inner latch at the induction variable increment instruction. Since now we have split at the inner latch branch instruction and have properly duplicated instructions over to the split block, we remove this limitation. Please refer to the test case updates to see how we now interchange loops where instructions exist between the induction variable increment and the branch instruction. Reviewed By: bmahjour Differential Revision: https://reviews.llvm.org/D115238
-
Michał Górny authored
Include the complete list of threads of all running processes in the FreeBSDKernel plugin. This makes it possible to inspect the states (including partial register dumps from PCB) of all kernel and userspace threads at the time of crash, or at the time of reading /dev/mem first. Differential Revision: https://reviews.llvm.org/D116255
-
Nikolas Klauser authored
Remove using declarations in common_reference.compile.pass.cpp Reviewed By: Quuxplusone, Mordante, #libc, jloser Spies: jloser, libcxx-commits Differential Revision: https://reviews.llvm.org/D116744
-
Nico Weber authored
This reverts commit afdc6a0b. Breaks check-lld, see e.g.: https://lab.llvm.org/buildbot/#/builders/123/builds/8100/steps/8/logs/stdio
-
Vincent Lee authored
One of our internal arm64 apps hit a thunk out of range error when building with LLD. Per the comment, I'm arbitrarily increasing slop size to 256. Reviewed By: #lld-macho, thakis Differential Revision: https://reviews.llvm.org/D116705
-
Matthias Springer authored
The old function names (e.g., `replaceOp`) could have been confusing to users because they sound similar to rewriter functions, but have slightly different semantics. Differential Revision: https://reviews.llvm.org/D116449
-
Carlos Galvez authored
Currently, it's inconsistent that warnings are disabled if they come from system headers, unless they come from macros. Typically a user cannot act upon these warnings coming from system macros, so clang-tidy should ignore them unless the user specifically requests warnings from system headers via the corresponding configuration. This change broke the ProTypeVarargCheck check, because it was checking for the usage of va_arg indirectly, expanding it (it's a system macro) to detect the usage of __builtin_va_arg. The check has been fixed by checking directly what the rule is about: "do not use va_arg", by adding a PP callback that checks if any macro with name "va_arg" is expanded. The old AST matcher is still kept for compatibility with Windows. Add unit test that ensures warnings from macros are disabled when not using the -system-headers flag. Document the change in the Release Notes. Differential Revision: https://reviews.llvm.org/D116378
-
mydeveloperday authored
https://github.com/llvm/llvm-project/issues/53008 ``` template <class Id> using A = quantity /**/<kind<Id>, 1>; ``` the presence of the comment between identifier and template opener seems to be causing the qualifier alignment to fail Reviewed By: curdeius Fixes: #53008 Differential Revision: https://reviews.llvm.org/D116726
-
Arthur O'Dwyer authored
In the test files, replace the old-style tests with a simple static_assert, matching the current style as depicted in e.g. `ranges_uninitialized_default_construct.pass.cpp`. Preserve `is_function_like` (but renamed to `is_niebloid`) at ldionne's request. The removal of this test helper will happen in D116570 if at all. Differential Revision: https://reviews.llvm.org/D116384
-
Alexey Bataev authored
No need to include the order of the scalars beeing used as part of the alternate vectorization into account when trying to reorder the whole graph. Such elements better to reorder in the following phase because the subtree still ends up in shuffle. Part of D116688, fixes the regression in D116690. Differential Revision: https://reviews.llvm.org/D116740
-
Alexey Bataev authored
There is a bug in the reordering analysis stage. If the element with the given hash is not added to the map but has the same number of APOs and instructions with same parent, but different instruction opcode, it will be initalized with default values and then the counter is increased by 1. But the lane is not updated and default to 0 instead of the actual `Lane` value. It leads to the fact that the analysis is useless in many cases and default to lane 0 instead of actual lane with the minimum amount of APO operands. Differential Revision: https://reviews.llvm.org/D116690
-