- Feb 20, 2020
-
-
Hideto Ueno authored
Summary: As mentioned in D71974, it is useful for must-be-executed-context to explore CFG backwardly. This patch is ported from parts of D64975. We use a dominator tree to find the previous context if a dominator tree is available. Reviewers: jdoerfert, hfinkel, baziotis, sstefan1 Reviewed By: jdoerfert Subscribers: hiraditya, llvm-commits Tags: #llvm Differential Revision: https://reviews.llvm.org/D74817
-
Johannes Doerfert authored
-
Johannes Doerfert authored
-
Johannes Doerfert authored
We can look through calls with `returned` argument attributes when we collect subsuming positions. This allows us to get existing attributes from more places.
-
Johannes Doerfert authored
-
Matt Arsenault authored
This was checking for default operands in the current DAG instruction, rather than the correct result operand list. I'm not entirly sure how this managed to work before, but was failing for me when multiple default operands were overridden.
-
Johannes Doerfert authored
We are often interested in an assumed constant and sometimes it has to be an integer constant. Before we only looked for the latter, now we can ask for either.
-
Johannes Doerfert authored
If we propagate function pointers across function boundaries we can create new call edges. These need to be represented in the CG if we run as a CGSCC pass. In the new pass manager that is currently not handled by the CallGraphUpdater so we need to prevent the situation for now.
-
Jonas Devlieghere authored
As pointed out on lldb-commits this skipIfRemote is the better fit for the decorator.
-
Johannes Doerfert authored
We usually will ask for liveness of an argument anyway so we ended up lazily creating the attribute anyway. However, that is not always the case and even if it is we should go the eager route here. Various tests show how this can improve the outcome. One test exposed a problem with type mismatches between argument and call site argument, a fix is included. For liveness various more tests were added as well.
-
Lang Hames authored
-
Lang Hames authored
-
Johannes Doerfert authored
If a function pointer is casted into a different type the resulting expression can be a constant. If so, it can be used multiple times which cannot be handled by the AbstractCallSite constructor alone. Instead, we follow the cast expression uses now explicitly during the call site traversal.
-
Douglas Yung authored
The change added a test that required exceptions, so enable that explicitly so that it works on platforms that default to having exceptions disabled (like the PS4).
-
Sourabh Singh Tomar authored
-
Igor Kudrin authored
The comment described a linked CIE to be acquired lazily. That is not true and looks like it was never true. Differential Revision: https://reviews.llvm.org/D74761
-
Igor Kudrin authored
The CIE pointer field of an FDE record contains an offset to a corresponding CIE record. In object files, this value comes with relocation because the value has to be fixed when a linker combines the final section from multiple sources. In most object files there is only one CIE record at offset 0 of the .debug_frame section, so reading a relocated or a raw value makes no difference. However, in partially linked object files there are multiple CIE records and the relocations should be applied to recover the right offset value. Differential Revision: https://reviews.llvm.org/D74612
-
Nico Weber authored
-
Sam Clegg authored
Differential Revision: https://reviews.llvm.org/D74110
-
Jonas Devlieghere authored
These tests are not configured to run on the device.
-
River Riddle authored
A few tests are broken, but this allows for MLIR to build.
-
Sam Clegg authored
The changes the in-memory representation of wasm symbols such that their optional ImportName and ImportModule use llvm::Optional. ImportName is set whenever WASM_SYMBOL_EXPLICIT_NAME flag is set. ImportModule (for imports) is currently always set since it defaults to "env". In the future we can possibly extent to binary format distingish import which have explit module names. Tags: #llvm Differential Revision: https://reviews.llvm.org/D74109
-
Fangrui Song authored
-
Fangrui Song authored
C++ standard libraries have different random function implementations. We cannot expect a particular order.
-
Reid Kleckner authored
This has no effect on how LLVM passes the arguments, but it prevents rewriteWithInAlloca from thinking that these parameters should be part of the inalloca pack. Follow-up to D72114 Reviewed By: erichkeane Differential Revision: https://reviews.llvm.org/D74452
-
Hanhan Wang authored
Patch D74638 allows linalg.generic ops to use zero-rank shaped type operands, this also can be applied to linalg.indexed_generic ops.
-
Greg Clayton authored
Summary: The Offset provides the offset within the function in a SourceLocation struct. This allows us to show the byte offset within a function. We also track offsets within inline functions as well. Updated the lookup tests to verify the offset for functions and inline functions. 0x1000: main + 32 @ /tmp/main.cpp:45 Reviewers: labath, aadsm, serhiy.redko, jankratochvil, xiaobai, wallace, aprantl, JDevlieghere Subscribers: hiraditya, llvm-commits Tags: #llvm Differential Revision: https://reviews.llvm.org/D74680
-
Dan Liew authored
Summary: This substitution expands to the appropriate minimum deployment target flag where thread local storage (TLS) was first introduced on Darwin platforms. For all other platforms the substitution expands to an empty string. E.g. for macOS the substitution expands to `-mmacosx-version-min=10.12` This patch adds support for the substitution (and future substitutions) by doing a minor refactor and then uses the substitution in the relevant TSan tests. rdar://problem/59568956 Reviewers: yln, kubamracek, dvyukov, vitalybuka Subscribers: #sanitizers, llvm-commits Tags: #sanitizers, #llvm Differential Revision: https://reviews.llvm.org/D74802
-
Nathan James authored
Added FixItHint comments to ReservedIdentifierCheck and IdentifierNamingCheck to trick the python scripts into detecting a fix it is provided as it can't see the FixItHints in RenamerClangTidyCheck.cpp
-
Thomas Lively authored
Summary: The instruction at `DefI` can sometimes be destroyed by `rematerializeCheapDef`, so it should not be used after calling that function. The fix is to use `Insert` instead when examining additional multivalue stackifications. `Insert` is the address of the new defining instruction after all moves and rematerializations have taken place. Reviewers: aheejin Subscribers: dschuff, sbc100, jgravelle-google, hiraditya, sunfish, llvm-commits Tags: #llvm Differential Revision: https://reviews.llvm.org/D74875
-
Alexander Lanin authored
Summary: This fixes https://bugs.llvm.org/show_bug.cgi?id=39310 Reviewers: malcolm.parsons, ioeric Reviewed By: malcolm.parsons Subscribers: xazax.hun Tags: #clang-format, #clang-tools-extra Differential Revision: https://reviews.llvm.org/D74800
-
LLVM GN Syncbot authored
-
Lang Hames authored
-
Matt Arsenault authored
We probably want this, and I've meant to turn this on for a long time. SC actually emits a special case to early-out for a 1 denominator, which perhaps should also be considered.
-
Yaxun (Sam) Liu authored
Differential Revision: https://reviews.llvm.org/D74807
-
Matt Arsenault authored
-
Matt Arsenault authored
Use common check prefix, although update_mir_test_checks makes this unnecessarily annoying. Also make sure to have uses in case that ever ends up mattering.
-
Lang Hames authored
-
Lang Hames authored
-
David Goldman authored
Forgot to update lines for RUNs
-