[llvm] 3533472 - [CostModel][X86] Remove comments that begin with a filecheck prefix.

Simon Pilgrim via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 13 10:39:36 PDT 2020


Author: Simon Pilgrim
Date: 2020-04-13T18:39:24+01:00
New Revision: 353347288bcf78c15a76f6fcf54ece43671a1f14

URL: https://github.com/llvm/llvm-project/commit/353347288bcf78c15a76f6fcf54ece43671a1f14
DIFF: https://github.com/llvm/llvm-project/commit/353347288bcf78c15a76f6fcf54ece43671a1f14.diff

LOG: [CostModel][X86] Remove comments that begin with a filecheck prefix.

Stop filecheck from confusing a general comment with a check.

Added: 
    

Modified: 
    llvm/test/Analysis/CostModel/X86/vselect-cost.ll

Removed: 
    


################################################################################
diff  --git a/llvm/test/Analysis/CostModel/X86/vselect-cost.ll b/llvm/test/Analysis/CostModel/X86/vselect-cost.ll
index 582942355692..f945764402bc 100644
--- a/llvm/test/Analysis/CostModel/X86/vselect-cost.ll
+++ b/llvm/test/Analysis/CostModel/X86/vselect-cost.ll
@@ -146,8 +146,8 @@ define i32 @test_select() {
   ret i32 undef
 }
 
-; SSE41 added blend instructions with an immediate for <2 x double> and
-; <4 x float>. Integers of the same size should also use those instructions.
+; Immediate blend instructions for <2 x double> and <4 x float> added at SSE41.
+; Integers of the same size should also use those instructions.
 
 define <2 x i64> @test_2i64(<2 x i64> %a, <2 x i64> %b) {
 ; SSE2-LABEL: 'test_2i64'
@@ -234,8 +234,9 @@ define <16 x i8> @test_16i8(<16 x i8> %a, <16 x i8> %b) {
   ret <16 x i8> %sel
 }
 
-; AVX added blend instructions with an immediate for <4 x double> and
-; <8 x float>. Integers of the same size should also use those instructions.
+; Immediate blend instructions for <4 x double> and <8 x float> added at AVX.
+; Integers of the same size should also use those instructions.
+
 define <4 x i64> @test_4i64(<4 x i64> %a, <4 x i64> %b) {
 ; SSE2-LABEL: 'test_4i64'
 ; SSE2-NEXT:  Cost Model: Found an estimated cost of 6 for instruction: %sel = select <4 x i1> <i1 true, i1 false, i1 false, i1 true>, <4 x i64> %a, <4 x i64> %b
@@ -304,7 +305,6 @@ define <8 x float> @test_8float(<8 x float> %a, <8 x float> %b) {
   ret <8 x float> %sel
 }
 
-; AVX2
 define <16 x i16> @test_16i16(<16 x i16> %a, <16 x i16> %b) {
 ; SSE2-LABEL: 'test_16i16'
 ; SSE2-NEXT:  Cost Model: Found an estimated cost of 6 for instruction: %sel = select <16 x i1> <i1 true, i1 false, i1 false, i1 false, i1 true, i1 false, i1 false, i1 false, i1 true, i1 false, i1 false, i1 false, i1 true, i1 false, i1 false, i1 false>, <16 x i16> %a, <16 x i16> %b


        


More information about the llvm-commits mailing list