[llvm] r228525 - Correctly combine alias.scope metadata by a union instead of intersecting

[Hans Wennborg]

On Sun, Feb 8, 2015 at 7:06 PM, Hal Finkel <hfinkel at anl.gov> wrote:
> ----- Original Message -----
>> From: "Hans Wennborg" <hans at chromium.org>
>> To: "Bjorn Steinbrink" <bsteinbr at gmail.com>, "Hal Finkel" <hfinkel at anl.gov>
>> Cc: "llvm-commits" <llvm-commits at cs.uiuc.edu>
>> Sent: Sunday, February 8, 2015 8:58:03 PM
>> Subject: Re: [llvm] r228525 - Correctly combine alias.scope metadata by a union instead of intersecting
>>
>> Hal,
>>
>> Björn marked this as a 3.6 regression in PR22507. Is this commit okay
>> for merging?
>
> Yes, I think that should be fine.

Merged in r228561.

Cheers,
Hans

>> On Sun, Feb 8, 2015 at 9:07 AM, Bjorn Steinbrink <bsteinbr at gmail.com>
>> wrote:
>> > Author: bsteinbr
>> > Date: Sun Feb  8 11:07:14 2015
>> > New Revision: 228525
>> >
>> > URL: http://llvm.org/viewvc/llvm-project?rev=228525&view=rev
>> > Log:
>> > Correctly combine alias.scope metadata by a union instead of
>> > intersecting
>> >
>> > Summary:
>> > The alias.scope metadata represents sets of things an instruction
>> > might
>> > alias with. When generically combining the metadata from two
>> > instructions the result must be the union of the original sets,
>> > because
>> > the new instruction might alias with anything any of the original
>> > instructions aliased with.
>>
>
> --
> Hal Finkel
> Assistant Computational Scientist
> Leadership Computing Facility
> Argonne National Laboratory