[all-commits] [llvm/llvm-project] caa755: [BasicAA] Make BasicAA a cfg pass.

[calixteman via All-commits]

  Branch: refs/heads/release/10.x
  Home:   https://github.com/llvm/llvm-project
  Commit: caa755f90e3711443180adec834fd544e3646e2e
      https://github.com/llvm/llvm-project/commit/caa755f90e3711443180adec834fd544e3646e2e
  Author: Alina Sbirlea <asbirlea at google.com>
  Date:   2020-06-23 (Tue, 23 Jun 2020)

  Changed paths:
    M llvm/lib/Analysis/BasicAliasAnalysis.cpp
    M llvm/test/Analysis/BasicAA/phi-values-usage.ll
    M llvm/test/Other/opt-O2-pipeline.ll
    M llvm/test/Other/opt-O3-pipeline.ll
    M llvm/test/Other/opt-Os-pipeline.ll

  Log Message:
  -----------
  [BasicAA] Make BasicAA a cfg pass.

Summary:
Part of the changes in D44564 made BasicAA not CFG only due to it using
PhiAnalysisValues which may have values invalidated.
Subsequent patches (rL340613) appear to have addressed this limitation.

BasicAA should not be invalidated by non-CFG-altering passes.
A concrete example is MemCpyOpt which preserves CFG, but we are testing
it invalidates BasicAA.

llvm-dev RFC: https://groups.google.com/forum/#!topic/llvm-dev/eSPXuWnNfzM

Reviewers: john.brawn, sebpop, hfinkel, brzycki

Subscribers: hiraditya, llvm-commits

Tags: #llvm

Differential Revision: https://reviews.llvm.org/D74353

(cherry picked from commit 0cecafd647ccd9d0acc5968d4d6e80c1cbdee275)


  Commit: 5ccc104c206c02e712dec68cc0daef6379f51861
      https://github.com/llvm/llvm-project/commit/5ccc104c206c02e712dec68cc0daef6379f51861
  Author: Calixte Denizet <calixte.denizet at gmail.com>
  Date:   2020-06-23 (Tue, 23 Jun 2020)

  Changed paths:
    M compiler-rt/test/profile/Inputs/instrprof-gcov-multithread_fork.cpp

  Log Message:
  -----------
  [compiler-rt] Reduce the number of threads in gcov test to avoid failure

Summary:
Patch in D78477 introduced a new test for gcov and this test is failing on arm:
 - http://lab.llvm.org:8011/builders/clang-cmake-thumbv7-full-sh/builds/4752/steps/ninja%20check%202/logs/stdio
  - http://lab.llvm.org:8011/builders/clang-cmake-armv7-full/builds/10501/steps/ninja%20check%202/logs/stdio
So try to fix it in reducing the number of threads.

Reviewers: marco-c

Reviewed By: marco-c

Subscribers: dberris, kristof.beyls, #sanitizers, serge-sans-paille, sylvestre.ledru

Tags: #sanitizers

Differential Revision: https://reviews.llvm.org/D79621

(cherry picked from commit 0da37bedc2667da371eda30595a06210595881d0)


Compare: https://github.com/llvm/llvm-project/compare/1f3be0e9b7b4...5ccc104c206c