[cfe-commits] [PATCH][Review Please] Devirtualize record type member object

[Yin Ma]

Hi Richard,

 

     What test cases you need? Do you mean that a test sits in clang test
folder

With a code that can be devirtualized?

 

Thanks,

 

                          Yin 

 

From: metafoo at gmail.com [mailto:metafoo at gmail.com] On Behalf Of Richard
Smith
Sent: Monday, July 30, 2012 1:13 PM
To: Yin Ma
Cc: cfe-commits at cs.uiuc.edu
Subject: Re: [cfe-commits] [PATCH][Review Please] Devirtualize record type
member object

 

On Mon, Jul 30, 2012 at 10:54 AM, Yin Ma <yinma at codeaurora.org> wrote:

Hi,

Currently, the LLVM devirtualization forgets to handle a simple code pattern
where can be devirutalized. The code pattern is when a call is from the
record type member object. Currently, LLVM only supports devirtualizing a
call from the record type object defined in the same function. If object is
defined as member, it should be supported. The patch is to enable
devirtualization for this case.

In the attachment.
devir.diff is the code change
the rest of files are the results from validation tests

Please give a review.

The patch looks correct, but needs some test cases added before it can be
checked in.

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20120730/b6264c73/attachment.html>