[PATCH] D62229: [TargetTransformInfo] early return on nullptr

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 22 04:12:23 PDT 2019


fhahn added a comment.

I am pretty sure from the uses of getGEPCost that Ptr should never be null. It is also used without check later on. I think it would be better to drop the `if (Ptr != nullptr)` check.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D62229





More information about the llvm-commits mailing list