On 11-07-11 18:53 , Delesley Hutchins wrote:
This patch fixes get_canonical_lock_expr so that it works on lock
expressions that involve a MEM_REF.  Gimple code can use either
MEM_REF or INDIRECT_REF in many expressions, and the choice of which
to use is somewhat arbitrary.  The canonical form of a lock expression
must rewrite all MEM_REFs to INDIRECT_REFs to accurately compare
expressions.  The surrounding "if" block prevented this rewrite from
happening in certain cases.

Bootstrapped and passed GCC regression testsuite on x86_64-unknown-linux-gnu.

Okay for branches/annotalysis and google/main?

Remember

  -DeLesley


2011-07-06   DeLesley Hutchins<deles...@google.com>
       * cp_get_virtual_function_decl.c (handle_call_gs): Changes
       function to return null if the method cannot be found.

>   2011-07-11   DeLesley Hutchins<deles...@google.com>
>         * tree-threadsafe-analyze.c (tree-threadsafe-analyze.c)
>          Changed to force rewrite on MEM_REF

You keep forgetting a blank line after the date line :)
Needs a ':' after ')', and the sentence should end in '.'.

The change description does not seem to be related to the change in the code.

OK, otherwise.


Diego.

Reply via email to