[PATCH] D34056: Tail merge size

[Sanjay Patel via Phabricator via llvm-commits]

spatel added inline comments.


================
Comment at: test/CodeGen/X86/loop-search.ll:2
+; NOTE: Assertions have been autogenerated by utils/update_llc_test_checks.py
 ; NOTE: Assertions have been autogenerated by utils/update_test_checks.py
 ; RUN: llc < %s -mtriple=x86_64-apple-darwin | FileCheck %s
----------------
RKSimon wrote:
> Remove this line - we should only be generating with update_llc_test_checks.py
I changed this (and several other test files) with rL305206, and I've specialized update_test_checks.py to only work with  'opt' to avoid this problem in the future (rL305208). 

We could rename that script to have 'opt' in its name too, but that might be more trouble than its worth because we now have a ton of IR tests with the existing script name in a comment line as seen here.


https://reviews.llvm.org/D34056