[PATCH] D65118: Analysis: Don't look through aliases when simplifying GEPs.

Johannes Doerfert via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 22 19:37:08 PDT 2019


jdoerfert reopened this revision.
jdoerfert added a comment.
This revision is now accepted and ready to land.

I saw this and I had a thought, please let me know what you think:

I would assume this issue is present in many more contexts. Maybe I am mistaken though.
If I am not, it is probably because we look through *any* alias by default without considering
the linker. Maybe, the right "fix" is to look through "exact definition" aliases only (by default).

(I reopened this because I do not know if Phab will otherwise send out emails).


Repository:
  rL LLVM

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

https://reviews.llvm.org/D65118





More information about the llvm-commits mailing list