[PATCH] D83576: [BasicAA] Fix -basicaa-recphi for geps with negative offsets

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Jul 19 12:38:58 PDT 2020


nikic added a comment.

@xbolva00 This is an artifact I've noticed a few times. When this code in BasicAA is changed, there is a measurable compile-time impact even for changes that should be NFC. I suspect that it triggers some inlining heuristic in a visible way, but never looked into it. I don't think there's anything problematic in this commit itself (it changes functionality that is currently disabled.)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D83576





More information about the llvm-commits mailing list