[PATCH] D150464: [FuncSpec] Improve the accuracy of the cost model.

[Vincent Lee via Phabricator via llvm-commits]

thevinster added subscribers: smeenai, thevinster.
thevinster added a comment.

@labrinea This change hits a crash when trying to build lld with thinLTO (i.e. stage2 lld) and with assertions enabled. I verified that the revision before this change works. I get the following assertion failure

  opt: <workspace>/llvm-project/llvm/include/llvm/Support/Casting.h:578: decltype(auto) llvm::cast(From *) [To = llvm::ConstantInt, From = llvm::Constant]: Assertion `isa<To>(Val) && "cast<Ty>() argument of incompatible type!"' failed.

I reduced the issue and this can be reproed by building opt on this revision and then running

  $ opt reduced.ll -o /dev/null -O3

where reduced.ll is https://reviews.llvm.org/P8311

Could you please take a look and advise the path forward?

cc: @smeenai


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D150464