[PATCH] D100353: Support optnone in SCCP

[Paul Robinson via Phabricator via llvm-commits]

probinson added a comment.

IIRC, the exception for IPOs was because we hadn't thought too much about what optnone ought to mean in that context--specifically, when two functions would interact in an IPO-ish way, but one function has optnone and  the other doesn't.

I think it's fine to respecify optnone in a way that lets it factor into IPO decisions while preserving the intent of optnone (which is that we should leave them alone as much as possible).  But, we should actually respecify it that way; then IPOs that don't conform become bugs/TODOs that need taking care of.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D100353