http://llvm.org/bugs/show_bug.cgi?id=2577

Chandler Carruth <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|REOPENED                    |RESOLVED
                 CC|                            |[email protected]
         Resolution|                            |FIXED

--- Comment #7 from Chandler Carruth <[email protected]> 2011-10-19 05:40:37 
CDT ---
Fixed with the following commits:

r142168 -- fix most egregious corruption of probability metadata
r142491 -- add testing facility to ensure we actually get expect into
probabilities
r142492 -- add support for expects which feed branches to the probability
analysis
r142493 -- add support for expects which feed switches to the probability
analysis

We're now back to the state of 1) the probabilities haven't been completely
audited, and 2) we only have a few passes that take advantage of them. But that
was the desired state for closing this PR. __builtin_expect now influences code
generation.


Chris, you had expressed an interest in pulling these into 3.0 -- it seems
somewhat borderline to me though. I'll leave the decision to you and Bill to
sort out. =]

-- 
Configure bugmail: http://llvm.org/bugs/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
_______________________________________________
LLVMbugs mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/llvmbugs

Reply via email to