ChuanqiXu abandoned this revision.
ChuanqiXu added a comment.

Got your points. Let's postpone this one.

But I want to emphasize that this patch (and the thin PCM) will decrease the 
performance. While LTO can save the regression, LTO is not widely used. 
(ThinLTO can only mitigate this.) I mean the default behavior shouldn't cause 
performance regression. We can offer a new alternative  for the users but we 
can't enable that by default simply now.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D144844

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to