[llvm-commits] [llvm] r89398 - in /llvm/trunk: include/llvm/Analysis/CaptureTracking.h lib/Analysis/BasicAliasAnalysis.cpp lib/Analysis/CaptureTracking.cpp lib/Transforms/IPO/FunctionAttrs.cpp test/Transforms/GVN/nonescaping-malloc.ll

[Duncan Sands]

Hi,

> Duncan (IIRC) actually implemented this behaviour early on and we 
> removed it because it was likely to be slow and didn't seem to provide 
> any performance benefit at the time.

yes, IIRC, when run over the entire testsuite it resulted in three additional
functions being marked nocapture, and having these functions marked nocapture
resulted in no optimization improvements.  Since it was hard to do the analysis
correctly, I removed it as being more trouble than it was worth.

Ciao,

Duncan.