r174627 - FileCheckize volatile test
John McCall
rjmccall at apple.com
Thu Feb 7 17:21:19 PST 2013
On Feb 7, 2013, at 7:39 AM, Tim Northover <tim.northover at arm.com> wrote:
> Author: tnorthover
> Date: Thu Feb 7 09:39:16 2013
> New Revision: 174627
>
> URL: http://llvm.org/viewvc/llvm-project?rev=174627&view=rev
> Log:
> FileCheckize volatile test
I appreciate the thought, but changing it from just counting the number of times
'volatile' shows up to, essentially, putting a minimum on the number of times
'volatile load' shows up is not a huge improvement. Would you mind
further improving this test to (1) actually match the operands and (2) do the
negative testing for the lines which we expect to produce non-volatile loads?
John.
More information about the cfe-commits
mailing list