- Mar 16, 2022
-
-
Joe Nash authored
NFC. Hasn't been updated since the update script started adding check-next. Reviewed By: arsenm Differential Revision: https://reviews.llvm.org/D121719
-
Aart Bik authored
Reviewed By: bixia Differential Revision: https://reviews.llvm.org/D121660
-
Roman Lebedev authored
While `-march=` is correctly detected as `znver3` for the cpu, apparently the model check is incorrect: ``` $ lscpu Architecture: x86_64 CPU op-mode(s): 32-bit, 64-bit Address sizes: 48 bits physical, 48 bits virtual Byte Order: Little Endian CPU(s): 32 On-line CPU(s) list: 0-31 Vendor ID: AuthenticAMD Model name: AMD Ryzen 9 5950X 16-Core Processor CPU family: 25 Model: 33 Thread(s) per core: 2 Core(s) per socket: 16 Socket(s): 1 Stepping: 0 Frequency boost: disabled CPU max MHz: 6017.8462 CPU min MHz: 2200.0000 BogoMIPS: 8050.07 Flags: fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov pat pse36 clflush mmx fxsr sse sse2 ht syscall nx mmxext fxsr_opt pdpe1gb rdtscp lm constant_tsc rep_good nopl nonstop_tsc cpuid extd_apicid aperfmperf rapl pni pclmulqdq monitor ssse 3 fma cx16 sse4_1 sse4_2 movbe popcnt aes xsave avx f16c rdrand lahf_lm cmp_legacy svm extapic cr8_legacy abm sse4a misalignsse 3dnowprefetch osvw ibs skinit wdt tce topoext perfctr_core perfctr_nb bpext perfctr_llc mwaitx cpb cat_l3 cdp_l3 hw_p state ssbd mba ibrs ibpb stibp vmmcall fsgsbase bmi1 avx2 smep bmi2 erms invpcid cqm rdt_a rdseed adx smap clflushopt clwb sha_ni xsaveopt xsavec xgetbv1 xsaves cqm_llc cqm_occup_llc cqm_mbm_total cqm_mbm_local clzero irperf xsaveerptr rdpru wbn oinvd arat npt lbrv svm_lock nrip_save tsc_scale vmcb_clean flushbyasid decodeassists pausefilter pfthreshold avic v_vmsave_vmload vgif v_spec_ctrl umip pku ospke vaes vpclmulqdq rdpid overflow_recov succor smca fsrm Virtualization features: Virtualization: AMD-V Caches (sum of all): L1d: 512 KiB (16 instances) L1i: 512 KiB (16 instances) L2: 8 MiB (16 instances) L3: 64 MiB (2 instances) NUMA: NUMA node(s): 1 NUMA node0 CPU(s): 0-31 Vulnerabilities: Itlb multihit: Not affected L1tf: Not affected Mds: Not affected Meltdown: Not affected Spec store bypass: Mitigation; Speculative Store Bypass disabled via prctl Spectre v1: Mitigation; usercopy/swapgs barriers and __user pointer sanitization Spectre v2: Mitigation; Retpolines, IBPB conditional, IBRS_FW, STIBP always-on, RSB filling Srbds: Not affected Tsx async abort: Not affected ``` Model is 33 (0x21), while the code was expecting it to be `0x00 .. 0x1F`. https://github.com/torvalds/linux/blob/v5.17-rc8/drivers/hwmon/k10temp.c#L432-L453 agrees. I'm not sure if other ranges listed here should also be accepted. I noticed this while implementing CPU model detection for halide (https://github.com/halide/Halide/pull/6648) Reviewed By: craig.topper Differential Revision: https://reviews.llvm.org/D121708 -
Yi Kong authored
This follows the same warning GCC produces. Differential Revision: https://reviews.llvm.org/D121683
-
Joe Nash authored
-
Jonas Devlieghere authored
This reverts commit 242c574d because it breaks the following tests on the bots: - TestGuiExpandThreadsTree.py - TestBreakpointCallbackCommandSource.py
-
Igor Kudrin authored
Template functions share the same lines in source files, so the common container of lines' properties cannot be used to calculate the coverage statistics of individual functions. > cat tmpl.cpp template <int N> int test() { return N; } int main() { return test<1>() + test<2>(); } > clang++ --coverage tmpl.cpp -o tmpl > ./tmpl > llvm-cov gcov tmpl.cpp -f ... Function '_Z4testILi1EEiv' Lines executed:100.00% of 1 Function '_Z4testILi2EEiv' Lines executed:-nan% of 0 ... > llvm-cov-patched gcov tmpl.cpp -f ... Function '_Z4testILi1EEiv' Lines executed:100.00% of 1 Function '_Z4testILi2EEiv' Lines executed:100.00% of 1 ... Differential Revision: https://reviews.llvm.org/D121390 -
Shafik Yaghmour authored
Currently DW_OP_deref_size just drops the ValueType::FileAddress case and does not attempt to handle it. This adds support for this case and a test that verifies this support. I did a little refactoring since DW_OP_deref and DW_OP_deref_size have some overlap in code. Also see: rdar://66870821 Differential Revision: https://reviews.llvm.org/D121408
-
Andrzej Warzynski authored
This patch adds support for lowering of the `ibits` intrinsic from Fortran to the FIR dialect of MLIR. This is part of the upstreaming effort from the `fir-dev` branch in [1]. [1] https://github.com/flang-compiler/f18-llvm-project Differential Revision: https://reviews.llvm.org/D121693 Co-authored-by:
Jean Perier <jperier@nvidia.com> Co-authored-by:
Valentin Clement <clementval@gmail.com> Co-authored-by:
V Donaldson <vdonaldson@nvidia.com>
-
Jonas Devlieghere authored
I thought that x86GetSupportedArchitectures would always return x86_64-apple-macosx as a compatible architecture, regardless of the host achitecture, but the Debian bot disagrees with that.
-
Jonas Devlieghere authored
Add synchronization to the IOHandler to prevent multiple threads from writing concurrently to the output or error stream. A scenario where this could happen is when a thread (the default event thread for example) is using the debugger's asynchronous stream. We would delegate this operation to the IOHandler which might be running on another thread. Until this patch there was nothing to synchronize the two at the IOHandler level. Differential revision: https://reviews.llvm.org/D121500
-
Andrzej Warzynski authored
This patch adds support for lowering of the `dim` intrinsic from Fortran to the FIR dialect of MLIR. This is part of the upstreaming effort from the `fir-dev` branch in [1]. [1] https://github.com/flang-compiler/f18-llvm-project Differential Revision: https://reviews.llvm.org/D121689 Co-authored-by:
Jean Perier <jperier@nvidia.com> Co-authored-by:
Eric Schweitz <eschweitz@nvidia.com> Co-authored-by:
Valentin Clement <clementval@gmail.com> Co-authored-by:
V Donaldson <vdonaldson@nvidia.com>
-
Siva Chandra Reddy authored
Reviewed By: lntue Differential Revision: https://reviews.llvm.org/D121676
-
Jonas Devlieghere authored
This patch is another attempt to fix platform selection on Apple Silicon. It partially undoes D117340 which tried to fix the issue by always instantiating a remote-ios platform for "iPhone and iPad Apps on Apple Silicon Macs". While the previous patch worked for attaching, it broke launching and everything else that expects the remote platform to be connected. I made an attempt to work around that, but quickly found out that there were just too may places that had this assumption baked in. This patch takes a different approach and reverts back to marking the host platform compatible with iOS triples. This brings us back to the original situation where platform selection was broken for remote iOS debugging on Apple Silicon. To fix that, we now look at the process' host architecture to differentiate between iOS binaries running remotely and iOS binaries running locally. I tested the following scenarios, which now all uses the desired platform: - Launching an iOS binary on macOS: uses the host platform - Attaching to an iOS binary on macOS: uses the host platform - Attaching to a remote iOS binary: uses the remote-ios platform rdar://89840215 Differential revision: https://reviews.llvm.org/D121444
-
- Mar 15, 2022
-
-
Andrzej Warzynski authored
This patch adds support for lowering the `dot_product` intrinsic from Fortran to the FIR dialect of MLIR. This is part of the upstreaming effort from the `fir-dev` branch in [1]. [1] https://github.com/flang-compiler/f18-llvm-project Differential Revision: https://reviews.llvm.org/D121684 Co-authored-by:
Jean Perier <jperier@nvidia.com> Co-authored-by:
Eric Schweitz <eschweitz@nvidia.com> Co-authored-by:
Valentin Clement <clementval@gmail.com> Co-authored-by:
Mark Leair <leairmark@gmail.com>
-
Bixia Zheng authored
Define IndexExpr before IndexVar. This is to prepare for the next change to support the use of index values in tensor expressions. Reviewed By: aartbik Differential Revision: https://reviews.llvm.org/D121649
-
Craig Topper authored
[LegalizeTypes][RISCV][WebAssembly] Expand ABS in PromoteIntRes_ABS if it will expand to sra+xor+sub later. If we promote the ABS and then Expand in LegalizeDAG, then both the sra and the xor will have their inputs sign extended. This generates extra code on RISCV which lacks an i8 or i16 sign extend instructon. If we expand during type legalization, then only the sra will get its input sign extended. RISCV is able to combine this with the sra by doing a shift left followed by an sra. Reviewed By: RKSimon Differential Revision: https://reviews.llvm.org/D121664
-
Craig Topper authored
[DAGCombiner][RISCV] Adjust (aext (and (trunc x), cst)) -> (and x, cst) to sext cst based on target preference RISCV strong prefers i32 values be sign extended to i64. This combine was always zero extending the constant using APInt methods. This adjusts the code so that it calls getNode using ISD::ANY_EXTEND instead. getNode will call TLI.isSExtCheaperThanZExt to decide how to handle the constant. Tests were copied from D121598 where I noticed that we were creating constants that were hard to materialize. Reviewed By: RKSimon Differential Revision: https://reviews.llvm.org/D121650
-
Pavel Labath authored
This reverts commit dddf4ce0. It breaks a couple of tests on macos.
-
Craig Topper authored
This code handles fixed vector SPLAT_VECTOR, but is never called in any tests. We only form fixed vector splat vectors for vXi64 on RV32 as part of DAGCombine. This will be type legalized to SPLAT_VECTOR_PARTS. So the Custom handling for SPLAT_VECTOR is never needed. This patch makes SPLAT_VECTOR for vXi64 'Legal' on RV32 so that DAGCombine will create it, but there's no need for Custom handler. It will still be type legalized to SPLAT_VECTOR_PARTS. Reviewed By: frasercrmck Differential Revision: https://reviews.llvm.org/D121673
-
Alex Brachet authored
-
Alex Brachet authored
Reviewed By: sivachandra Differential Revision: https://reviews.llvm.org/D121656
-
Yitzhak Mandelbaum authored
Terminators are handled specially in the transfer functions so we need an additional check on whether the analysis has disabled built-in transfer functions. Differential Revision: https://reviews.llvm.org/D121694
-
Sanjay Patel authored
This can be viewed as swapping the select arms: https://alive2.llvm.org/ce/z/jUvFMJ ...so we don't have the 'nsz' problem with the more general fold. This unlocks other folds for the motivating fabs example. This was discussed in issue #38828.
-
Sanjay Patel authored
-
Louis Dionne authored
Prior to this patch, there was no distinction between tests that check basic assertions and tests that check full-fledged iterator debugging assertions. Both were disabled when support for the debug mode is not provided in the dylib, which is stronger than it needs to be. Furthermore, all of the tests using "debug_macros.h" that contain more than one assertion in them were broken -- any code after the first assertion would never be executed. This patch refactors all of our assertion-related tests to: 1. Be enabled whenever they can, i.e. basic assertions tests are run even when the debug mode is disabled. 2. Use the superior `check_assertion.h` (previously `debug_mode_helper.h`) instead of `debug_macros.h`, which allows multiple assertions in the same program. 3. Coalesce some tests into the same file to make them more readable. 4. Use consistent naming for test files -- no more db{1,2,3,...,10} tests. This is a large but mostly mechanical patch. Differential Revision: https://reviews.llvm.org/D121462 -
Simon Moll authored
ISel for experimental.vp.strided.load|store for v256.32 types via lowering to vvp_load|store SDNodes. Reviewed By: kaz7 Differential Revision: https://reviews.llvm.org/D121616
-
Sam Carroll authored
When using `--convert-func-to-llvm=emit-c-wrappers` the attribute arguments of the wrapper would not be created correctly in some cases. This patch fixes that and introduces a set of tests for (hopefully) all corner cases. See https://github.com/llvm/llvm-project/issues/53503 Author: Sam Carroll <sam.carroll@lmns.com> Co-Author: Laszlo Kindrat <laszlo.kindrat@lmns.com> Reviewed By: ftynse Differential Revision: https://reviews.llvm.org/D119895
-
Tue Ly authored
Implement expm1f function that is correctly rounded for all rounding modes. This is based on expf implementation. From exhaustive testings, using expf implementation, and subtract 1.0 before rounding the final result to single precision gives correctly rounded results for all |x| > 2^-4 with 1 exception. When |x| < 2^-25, we use x + x^2 (implemented with a single fma). And for 2^-25 <= |x| <= 2^-4, we use a single degree-8 minimax polynomial generated by Sollya. Reviewed By: sivachandra, zimmermann6 Differential Revision: https://reviews.llvm.org/D121574
-
Simon Pilgrim authored
[InstCombine] Add general constant support to eq/ne icmp(add(X,C1),add(Y,C2)) -> icmp(add(X,C1-C2),Y) fold A further extension for Issue #32161 For eq/ne comparisons - the sign mismatch and bounds constraints are redundant, so if the that fold fails, fallback and just fold the constants directly. https://alive2.llvm.org/ce/z/cdodNQ The loop rotation test change looks mostly benign - the backend doesn't seem to suffer? https://gcc.godbolt.org/z/dErMY78To Differential Revision: https://reviews.llvm.org/D121551
-
Simon Pilgrim authored
Perform the mask inside parentheses before applying the offset
-
Ties Stuij authored
Reviewed By: amilendra Differential Revision: https://reviews.llvm.org/D121206
-
Arnamoy Bhattacharyya authored
Patch adds a new operation for the SIMD construct. The op is designed to be very similar to the existing `wsloop` operation, so that the `CanonicalLoopInfo` of `OpenMPIRBuilder` can be used. Reviewed By: shraiysh Differential Revision: https://reviews.llvm.org/D118065
-
Steven Wu authored
Fix darwin interface test after D121464. asan_rtl_x86_64.S is not available on Darwin. Reviewed By: kstoimenov Differential Revision: https://reviews.llvm.org/D121636
-
LLVM GN Syncbot authored
-
Wael Yehia authored
Revert "Load pass plugins during option processing, so that plugin options are registered and live." This reverts commit 5e8700ce.
-
Simon Pilgrim authored
Revert rG9c542a5a "Lower `@llvm.global_dtors` using `__cxa_atexit` on MachO" Mane of the build bots are complaining: Unknown command line argument '-lower-global-dtors'
-
Pavel Labath authored
Avoids polluting the namespace of all files including the header.
-
Wael Yehia authored
-