NNikita Popov[polly] Remove use of getWithSamePointeeType() (NFC)
| 文件 | 最后提交记录 | 最后更新时间 |
|---|---|---|
[polly] Remove use of getWithSamePointeeType() (NFC) | 3 年前 | |
[Polly] Remove Polly-ACC. Polly-ACC is unmaintained and since it has never been ported to the NPM pipeline, since D136621 it is not even accessible anymore without manually specifying the passes on the opt command line. Since there is no plan to put it to a maintainable state, remove it from Polly. Reviewed By: grosser Differential Revision: https://reviews.llvm.org/D142580 | 3 年前 | |
[Polly] Don't generate inter-iteration noalias metadata. This metadata was intended to mark all accesses within an iteration to be pairwise non-aliasing, in this case because every memory of a base pointer is touched (read or write) at most once. This is typical for 'sweeps' over all data. The stated motivation from D30606 is to ensure that unrolled iterations are considered non-aliasing. Rhe implemention had multiple issues: * The structure of the noalias metadata was malformed. D110026 added check in the verifier for this metadata, and the tests were failing since then. * This is not true for the outer loops of the BLIS matrix multiplication, where it was being inserted. Each element of A, B, C is accessed multiple times, as often as the loop not used as an index is iterating. * Scopes were added to SecondLevelOtherAliasScopeList (used for the !noalias scop list) on-the-fly when another SCEV was seen. This meant that previously visited instructions would not be updated with alias scopes that are only seen later, missing out those SCEVs they should not be aliasing with. * Since the !noalias scope list would ideally consists of all other SCEV for this base pointer, we might run quickly into scalability issues. Especially after unrolling there would probably at least once SCEV per instruction and unroll instance. * The inter-iteration noalias base pointer was not removed after leaving the loop marked with it, effectively marking everything after it to noalias as well. A solution I considered was to mark each instruction as non-aliasing with its own scope. The instruction itself would obviously alias itself, but such construction might also be considered invalid. Duplicating the instruction (e.g. due to speculation) would mark the instruction non-aliasing with its clone. I don't want to go into this territory, especially since the original motivation of determining unrolled instances as noalias based on SCEV is the what scev-aa does as well. This effectively reverts D30606 and D35761. | 4 年前 | |
[Polly] Remove Polly-ACC. Polly-ACC is unmaintained and since it has never been ported to the NPM pipeline, since D136621 it is not even accessible anymore without manually specifying the passes on the opt command line. Since there is no plan to put it to a maintainable state, remove it from Polly. Reviewed By: grosser Differential Revision: https://reviews.llvm.org/D142580 | 3 年前 | |
[Polly] Remove some bitcasts (NFC) No longer relevant with opaque pointers. | 3 年前 | |
Use *{Map,Set}::contains (NFC) | 3 年前 | |
[Polly] Remove some bitcasts (NFC) No longer relevant with opaque pointers. | 3 年前 | |
[Polly] Remove some bitcasts (NFC) No longer relevant with opaque pointers. | 3 年前 | |
[Polly] Remove some bitcasts (NFC) No longer relevant with opaque pointers. | 3 年前 | |
[NFC][TargetParser] Remove llvm/ADT/Triple.h I also ran git clang-format to get the headers in the right order for the new location, which has changed the order of other headers in two files. | 3 年前 | |
[Polly] Remove Polly-ACC. Polly-ACC is unmaintained and since it has never been ported to the NPM pipeline, since D136621 it is not even accessible anymore without manually specifying the passes on the opt command line. Since there is no plan to put it to a maintainable state, remove it from Polly. Reviewed By: grosser Differential Revision: https://reviews.llvm.org/D142580 | 3 年前 | |
Apply include-what-you-use #include removal suggestions. NFC. This removes unused includes (and forward declarations) as suggested by include-what-you-use. If a transitive include of a removed include is required to compile a file, I added the required header (or forward declaration if suggested by include-what-you-use). This should reduce compilation time and reduce the number of iterative recompilations when a header was changed. llvm-svn: 357209 | 7 年前 |