[PATCH] D49809: [InstCombine] Unlocked IO transformation - off by default

[Roman Lebedev via Phabricator via llvm-commits]

lebedev.ri added subscribers: efriedma, echristo, bkramer, lebedev.ri.
lebedev.ri added a comment.

Sadly, i'm not seeing much answers to the questions @vsk raised

> I'd like to see a fuller justification here. Could you elaborate?
> 
> - What does the typical improvement look like?
> - How common is this pattern is in the open source programs you looked at?
> - What do you think the policy should be for simplifying libcalls in a way that can break interceptors?
> 
>   Could you CC the reviewer, rja (Joe) on this thread? They may have some insight. I can't find their email address. They appear to have stopped commenting on Phab shortly after this commit landed.


Repository:
  rL LLVM

https://reviews.llvm.org/D49809