[PATCH] D115702: [LNT] Fix profile assigning to tests

[Thomas Preud'homme via Phabricator via llvm-commits]

thopre added inline comments.


================
Comment at: lnt/server/db/testsuitedb.py:1044
         for test_data in tests_data:
+            if len(test_data) == 2 and 'profile' in test_data:
+                # Ignore for now profile data without other metrics
----------------
slydiman wrote:
> thopre wrote:
> > If a profile is determined by having the name ends in .profile I'd prefer to check that explicitely (i.e. `test_data['name'].endswith('profie')`)
> It will not work because __init__.py (line 593) removes '.profile' from the name during preprocessing.
One more reason to rewrite this test.


Repository:
  rLNT LNT

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

https://reviews.llvm.org/D115702