[PATCH] D73705: [NFC][SCEV] Piping to pass new SCEVCheapExpansionBudget option into SCEVExpander::isHighCostExpansionHelper()

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 10 04:26:43 PST 2020


lebedev.ri added a comment.

In D73705#1866682 <https://reviews.llvm.org/D73705#1866682>, @mkazantsev wrote:

> Ok, thanks for clarification. Let them shoot their legs then. :) LGTM


Thank you for the review!
To be noted, i'm not vehemently opposed to hardening, but not before
there is a clear idea what we should be actually doing for the undefined cases.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D73705





More information about the llvm-commits mailing list