HsiangKai added a comment.

In D99741#2721669 <https://reviews.llvm.org/D99741#2721669>, @craig.topper 
wrote:

> In D99741#2721596 <https://reviews.llvm.org/D99741#2721596>, @thakis wrote:
>
>> In D99741#2721235 <https://reviews.llvm.org/D99741#2721235>, @craig.topper 
>> wrote:
>>
>>> I believe we had reductions at builds 7601 and 8175.
>>>
>>> It looks like there may have been an increase between builds 7979 and 7991 
>>> the build failed for a while there. I suspect "e951b04 
>>> <https://reviews.llvm.org/rGe951b045bfbf7974d66a4d5c040bbcf4d513c861> 
>>> [AArch64][SVE] Regression test all ACLE tests with C++" as it added a quite 
>>> a few RUN lines.
>>>
>>> I think we also caused an increase at build 8202 when these 4 commits went 
>>> in 
>>> bd32c2d 
>>> <https://reviews.llvm.org/rGbd32c2d64842c8d474bcce0e73dcd8918b3e0f2f> 
>>> [RISCV] Implement the vwcvt{u}.x.x.v/vncvt.x.x.w builtin.                   
>>>                                                                             
>>>                                                                             
>>>                                     
>>> 645c5f2 
>>> <https://reviews.llvm.org/rG645c5f27a84d795fc658b49c702d77cc6145bfec> 
>>> [RISCV] Implement the pseudo compare builtin.                               
>>>                                                                             
>>>                                                                             
>>>                                     
>>> bfb3fca 
>>> <https://reviews.llvm.org/rGbfb3fca7e110a367c9589495575913d83bea7095> 
>>> [RISCV] Implement the vfabs.v/vfneg.v builtin.                              
>>>                                                                             
>>>                                                                             
>>>                                     
>>> 4b24341 
>>> <https://reviews.llvm.org/rG4b2434172cc15ccd32c5c9110a3a4df18b8fba5e> 
>>> [RISCV] Implement the vmmv.m/vmnot.m builtin.
>>
>> Any disagreement about reverting riscv builtin changes until there's some 
>> way to test them without increasing check-clang time by 20%?
>
> We're preparing a patch to remove to stop testing both rv32 and rv64 on every 
> test. That should reduce the time by half. What is an acceptable number?

I have removed rv32 test cases for vector intrinsics in 
https://github.com/llvm/llvm-project/commit/b358a2be52480c008a0789c78f5df1bb2053bfd8.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D99741/new/

https://reviews.llvm.org/D99741

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to