- Dec 11, 2021
-
-
Jez Ng authored
1. After D113241, we have the section address easily accessible and no longer need to iterate across the LC_SEGMENT commands to emit LC_DATA_IN_CODE. 2. There's no need to store a pointer to the data in code entries during the parse step; we can just look it up as part of the output step. Reviewed By: #lld-macho, thakis Differential Revision: https://reviews.llvm.org/D115556
-
Jez Ng authored
... only whether they have more than zero. This simplifies the code slightly. I've also moved the field into the ConcatInputSection subclass since it doesn't actually get used by the other InputSections. Reviewed By: #lld-macho, oontvoo Differential Revision: https://reviews.llvm.org/D115539
-
Shilei Tian authored
-
Nico Weber authored
This reverts commit 9a3df8fb. 492de35d was reverted in 30fc88bf.
-
Sam McCall authored
-
Med Ismail Bennani authored
This reverts commit 492de35d. I tried to apply John's changes in 8d897ec9 that were expected to fix his patch but that didn't work unfortunately. Reverting this again to fix the macOS bots and leave him more time to investigate the issue.
-
Med Ismail Bennani authored
This reverts commit 8d897ec9. I tentativily landed John's patch in D115553 to fix the macOS bots but it's still failing as shown here: https://green.lab.llvm.org/green/job/lldb-cmake/39267/ Signed-off-by:
Med Ismail Bennani <medismail.bennani@gmail.com>
-
Sam McCall authored
Turns out these weren't obsolete after all... http://45.33.8.238/win/50653/step_9.txt
-
John Ericson authored
In 492de35d / D115544 I accidentally added ${LLVM_INCLUDE_DIR}/ to the destination path. This broke the "LLDB Incremental" build. Putting it back the way it was should fix it. Differencial Revision: https://reviews.llvm.org/D115553
-
Sam McCall authored
I started adding tests for all diags but found there were just too many cases. Differential Revision: https://reviews.llvm.org/D115484
-
Kirill Stoimenov authored
This reverts commit db124df4. Broke some builds: https://lab.llvm.org/buildbot/#/builders/98/builds/9895 https://lab.llvm.org/buildbot/#/builders/91/builds/434 Reviewed By: kstoimenov Differential Revision: https://reviews.llvm.org/D115564
-
Matt Arsenault authored
-
Jon Chesterfield authored
This reverts commit 7b9ab06d. Said code is better removed as part of a larger change.
-
Sam McCall authored
-
Lei Zhang authored
`(void)` was added when LogicalResult was marked as non discard. This commit cleans them up to properly propagate failures. Reviewed By: scotttodd Differential Revision: https://reviews.llvm.org/D115541
-
Lei Zhang authored
This should really come from a matching target environment. But as a default, it can be handy (to avoid always listing the full resource limits attribute in IR, etc.). It's common to see 32 so use that as the subgroup size. Reviewed By: scotttodd Differential Revision: https://reviews.llvm.org/D115534
-
Lei Zhang authored
It's legal per the Vulkan / SPIR-V spec; still it's better to avoid such duplication to have cleaner blob and reduce the binary size. Reviewed By: scotttodd Differential Revision: https://reviews.llvm.org/D115532
-
Lei Zhang authored
In SPIR-V, symbol names are encoded as `OpName` instructions. They are not semantic impacting and can be omitted, which can reduce the binary size. Reviewed By: scotttodd Differential Revision: https://reviews.llvm.org/D115531
-
Kirill Stoimenov authored
This change moves optimized callbacks from each .o file to compiler-rt. Reviewed By: vitalybuka, morehouse Differential Revision: https://reviews.llvm.org/D115396
-
Chia-hung Duan authored
bc657b2e adds the dependency to MemRefDialect. Reviewed By: GMNGeoffrey, mehdi_amini Differential Revision: https://reviews.llvm.org/D115558
-
Jayson Yan authored
Adds JSONScopedPrinter to llvm-readelf. It includes an empty JSONELFDumper class which will be used to override any LLVMELFDumper methods which utilize startLine() which JSONScopedPrinter cannot provide. This introduces a change where calls to llvm-readelf with non-ELF object files that specify --elf-output-style=GNU will now print file summary information where it previously didn't. Fixes previous Windows test failure which occured due to JSON escaping of '\' by not relying on LIT substitution. Reviewed By: jhenderson Differential Revision: https://reviews.llvm.org/D114225
-
Nicolas Vasilache authored
This reverts commit 5601821d. The prefix + canonical complete behavior is actually obsolete and should not be reintroduced. Reverting.
-
Andrew Browne authored
This reverts commit 2b554920. This change causes tsan test timeout on x86_64-linux-autoconf. The timeout can be reproduced by: git clone https://github.com/llvm/llvm-zorg.git BUILDBOT_CLOBBER= BUILDBOT_REVISION=eef8f3f8 llvm-zorg/zorg/buildbot/builders/sanitizers/buildbot_standard.sh
-
David Blaikie authored
-
Florian Mayer authored
Reviewed By: eugenis Differential Revision: https://reviews.llvm.org/D114294
-
Jon Roelofs authored
Differential Revision: https://reviews.llvm.org/D111158
-
Arjun P authored
The method that was previously used for computing dual variables was incorrect. This was used in the integer emptiness check algorithm, where this bug could lead to much longer running times. (Due to the way it is used, this never results in an incorrect emptiness check result.) This patch fixes the dual computation and adds some additional asserts that catch this bug, along with regression test cases that trigger the asserts when the incorrect dual computation is used. Reviewed By: Groverkss Differential Revision: https://reviews.llvm.org/D113803
-
Arjun P authored
Introduce a function `getNumIdKind` that returns the number of ids of the specified kind. Remove the function `assertAtMostNumIdKind` and instead just directly assert the inequality with a call to `getNumIdKind`.
-
Bogdan Graur authored
This reverts commit 847a6807. The reverted revision was causing miscompiles that manifest on AMD machines. Differential Revision: https://reviews.llvm.org/D115528
-
Thomas Raoux authored
Differential Revision: https://reviews.llvm.org/D115548
-
Uday Bondhugula authored
NFC. Move out and expose affine scalar replacement utility through affine utils. Renaming misleading forwardStoreToLoad -> affineScalarReplace. Update a stale doc comment. Differential Revision: https://reviews.llvm.org/D115495
-
Kirill Stoimenov authored
This is present in our assembly files. It should fix decorate_proc_maps.cpp failures because of shadow memory being allocated as executable. Reviewed By: vitalybuka Differential Revision: https://reviews.llvm.org/D115552
-
Nicolas Vasilache authored
InsertSliceOp may have subprefix semantics where missing trailing dimensions are automatically inferred directly from the operand shape. This revision fixes an overflow that occurs in such cases when the impl is based on the op rank. Differential Revision: https://reviews.llvm.org/D115549
-
Nico Weber authored
-
Jayson Yan authored
This reverts commit 824eef23. file-summary-json.test and pretty-print.test fails on Windows.
-
John Ericson authored
This reverts commit 797b50d4. See the original D99484. @mib who noticed the original problem could not longer reproduce it, after I tried and also failed. We are threfore hoping it went away on its own! Reviewed By: mib Differential Revision: https://reviews.llvm.org/D115544
-
Joseph Huber authored
This reverts commit 7c8f4e7b. Fails a few OpenMP tests, causes a few updates to segfault.
-
Jonathan Peyton authored
Allow filtering of resources based on core attributes. There are two new attributes added: 1) Core Type (intel_atom, intel_core) 2) Core Efficiency (integer) where the higher the efficiency, the more performant the core On hybrid architectures , e.g., Alder Lake, users can specify KMP_HW_SUBSET=4c:intel_atom,4c:intel_core to select the first four Atom and first four Big cores. The can also use the efficiency syntax. e.g., KMP_HW_SUBSET=2c:eff0,2c:eff1 Differential Revision: https://reviews.llvm.org/D114901
-
Michael Liao authored
-
Sami Tolvanen authored
Inline assembly refererences to static functions with ThinLTO+CFI were fixed in D104058 by creating aliases for promoted functions. Creating the aliases unconditionally resulted in an unexpected size increase in a Chrome helper binary: https://bugs.chromium.org/p/chromium/issues/detail?id=1261715 This is caused by the compiler being unable to drop unused code now referenced by the alias in module-level inline assembly. This change adds a .set_conditional assembly extension, which emits an assignment only if the target symbol is also emitted, avoiding phantom references to functions that could have otherwise been dropped. This is an alternative to the solution proposed in D112761. Reviewed By: pcc, nickdesaulniers, MaskRay Differential Revision: https://reviews.llvm.org/D113613
-