[PATCH] D51550: Use alias analysis to check for real interference in cascade comparison

Mon Sep 10 11:57:50 PDT 2018

christylee updated this revision to Diff 164712.
christylee added a comment.

For instructions that may have side effects, check that the instruction is a simple load or simple store before checking for alias.  For things like function calls, we would have to check that they don't throw and have no read/write to memory.  The function calls are potentially nested as well, it's probably better to just bail on those cases.

https://reviews.llvm.org/D51550

Files:
  lib/Transforms/Scalar/MergeICmps.cpp
  test/CodeGen/AArch64/O3-pipeline.ll
  test/CodeGen/Generic/llc-start-stop.ll
  test/CodeGen/X86/O3-pipeline.ll
  test/Transforms/MergeICmps/X86/alias-merge-blocks.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D51550.164712.patch
Type: text/x-patch
Size: 23402 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180910/59fb6de3/attachment.bin>