[all-commits] [llvm/llvm-project] cb7689: [SCEVExpander] Recognize urem idiom during expansi...

[Philip Reames via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: cb76896d6e45e2c9b7ef5e47b6ec37aeca43f7a8
      https://github.com/llvm/llvm-project/commit/cb76896d6e45e2c9b7ef5e47b6ec37aeca43f7a8
  Author: Philip Reames <preames at rivosinc.com>
  Date:   2024-06-19 (Wed, 19 Jun 2024)

  Changed paths:
    M llvm/lib/Analysis/ScalarEvolution.cpp
    M llvm/lib/Transforms/Utils/ScalarEvolutionExpander.cpp
    M llvm/test/Transforms/LoopStrengthReduce/X86/postinc-iv-used-by-urem-and-udiv.ll
    M llvm/test/Transforms/LoopVectorize/trip-count-expansion-may-introduce-ub.ll

  Log Message:
  -----------
  [SCEVExpander] Recognize urem idiom during expansion (#96005)

If we have a urem expression, emitting it as a urem is significantly
better that letting the fully expansion kick in. We have the risk of a
udiv or mul which could have previously been shared, but loosing that
seems like a reasonable tradeoff for being able to round trip a urem w/o
modification.



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications