[llvm] r280505 - Split the store of a wide value merged from an int-fp pair into multiple stores.

Wei Mi via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 2 18:57:01 PDT 2016


Sorry, I fix it at r280568. Thank you for reminding me the error. I am
surprised I didn't received any buildbot failure mail and thought the
commit went well. Is there anything wrong?

Wei.

On Fri, Sep 2, 2016 at 6:02 PM, Renato Golin <renato.golin at linaro.org> wrote:
> On 2 September 2016 at 18:17, Wei Mi via llvm-commits
> <llvm-commits at lists.llvm.org> wrote:
>> Author: wmi
>> Date: Fri Sep  2 12:17:04 2016
>> New Revision: 280505
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=280505&view=rev
>> Log:
>> Split the store of a wide value merged from an int-fp pair into multiple stores.
>>
>> For the store of a wide value merged from a pair of values, especially int-fp pair,
>> sometimes it is more efficent to split it into separate narrow stores, which can
>> remove the bitwise instructions or sink them to colder places.
>>
>> Now the feature is only enabled on x86 target, and only store of int-fp pair is
>> splitted. It is possible that the application scope gets extended with perf evidence
>> support in the future.
>
> Hi,
>
> This broke all buildbots that don't build X86 by default:
>
> http://lab.llvm.org:8011/builders/clang-cmake-aarch64-42vma/builds/11319
>
> http://lab.llvm.org:8011/builders/clang-cmake-aarch64-quick/builds/9894
>
> etc.
>
> Can you please move that test to an x86_64 specific directory or use
> the appropriate lit flags?
>
> http://llvm.org/docs/TestingGuide.html#platform-specific-tests
>
> cheers,
> --renato


More information about the llvm-commits mailing list