[all-commits] [llvm/llvm-project] c1d53a: [Attributor] Look through selects in genericValueT...

Johannes Doerfert via All-commits all-commits at lists.llvm.org
Sat Jul 10 14:49:39 PDT 2021


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: c1d53a316d6c7f7d80908ec8b5a65172f82e9721
      https://github.com/llvm/llvm-project/commit/c1d53a316d6c7f7d80908ec8b5a65172f82e9721
  Author: Johannes Doerfert <johannes at jdoerfert.de>
  Date:   2021-07-10 (Sat, 10 Jul 2021)

  Changed paths:
    M llvm/lib/Transforms/IPO/AttributorAttributes.cpp
    M llvm/test/Transforms/Attributor/lvi-for-ashr.ll
    M llvm/test/Transforms/Attributor/value-simplify.ll

  Log Message:
  -----------
  [Attributor] Look through selects in genericValueTraversal

If we can simplify the select condition we can avoid one value in the
traversal.

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


  Commit: 4761d29633ac7889329dc6ef966eab01c7b7903d
      https://github.com/llvm/llvm-project/commit/4761d29633ac7889329dc6ef966eab01c7b7903d
  Author: Johannes Doerfert <johannes at jdoerfert.de>
  Date:   2021-07-10 (Sat, 10 Jul 2021)

  Changed paths:
    M llvm/lib/Transforms/IPO/AttributorAttributes.cpp
    M llvm/test/Transforms/Attributor/value-simplify.ll

  Log Message:
  -----------
  [Attributor][FIX] Sanitize queries to LVI and ScalarEvolution

When we talk to outside analyse, e.g., LVI and ScalarEvolution, we need
to be careful with the query. The particular error occurred because we
folded a PHI node before the LVI query but the context location was now
not dominated by the value anymore. This is not supported by LVI so we
have to filter these situations before we query the outside analyses.


Compare: https://github.com/llvm/llvm-project/compare/5b05a5f6cee2...4761d29633ac


More information about the All-commits mailing list