[PATCH] D55009: [GVN] Don't perform scalar PRE on GEPs

Alexandros Lamprineas via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 4 09:07:57 PST 2018


labrinea marked an inline comment as done.
labrinea added inline comments.


================
Comment at: test/Transforms/GVN/PRE/pre-gep-load.ll:1
 ; NOTE: Assertions have been autogenerated by utils/update_test_checks.py
 ; RUN: opt < %s -basicaa -gvn -enable-load-pre -S | FileCheck %s
----------------
john.brawn wrote:
> The version of this file that I see in trunk doesn't have these autogenerated check lines. Is there intended to be a commit before this that adjusts the test?
Indeed. I've just added them in this revision to demonstrate the difference. I wasn't intending to do a separate commit unless it's necessary.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D55009/new/

https://reviews.llvm.org/D55009





More information about the llvm-commits mailing list