[PATCH] D93974: [ValueTracking] Safe assumption context for args

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Jan 2 02:24:46 PST 2021


nikic added a reviewer: nikic.
nikic added a comment.

Am I understanding correctly that this tries to use the last instruction in the entry block rather than the first one to avoid triggering the ephemeral value check, in case the first instruction is part of an assumption?

In any case, I don't think it's appropriate to perform a full block scan to determine the context instruction. safeCxtI() should be cheap (as in O(1)).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D93974



More information about the llvm-commits mailing list