Add a test for invalidation

[timholy]

xref JuliaSIMD/CPUSummary.jl#3 (comment)

[codecov]

Codecov Report

Base: 86.47% // Head: 86.47% // No change to project coverage 👍

Coverage data is based on head (2c3cbc7) compared to base (def5ad1).
Patch has no changes to coverable lines.

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #426   +/-   ##
=======================================
  Coverage   86.47%   86.47%           
=======================================
  Files          37       37           
  Lines        9306     9306           
=======================================
  Hits         8047     8047           
  Misses       1259     1259           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

[chriselrod]

Thanks, this will be useful!
It'll also run automatically for packages that include LoopVectorization.jl in their downstream tests.

[chriselrod]

@timholy maybe there's a better way to run tests in parallel, but mind updating the ci workflows?

LoopVectorization.jl/.github/workflows/ci.yml

Lines 36 to 41 in def5ad1

	loopvectorization_test:
	- part1
	- part2
	- part3
	- part4
	- part5

LoopVectorization.jl/.github/workflows/ci-julia-nightly.yml

Lines 35 to 40 in def5ad1

	loopvectorization_test:
	- part1
	- part2
	- part3
	- part4
	- part5

All you should have to do is add part6 to both of these.

[chriselrod]

  Activating project at `~/work/LoopVectorization.jl/LoopVectorization.jl/test/precompile`
ERROR: LoadError: ArgumentError: Package LoopVectorization [bdcacae8-1622-11e9-2a5c-532679323890] is required but does not seem to be installed:
 - Run `Pkg.instantiate()` to install all recorded dependencies.

@DilumAluthge what's the best way to make sure it finds the LV we're testing (or is there someone better to ask)?

[timholy]

Thanks for adding it to CI, sorry I didn't get to it first.

[DilumAluthge]

I would just add SnoopCompile and SnoopCompileCore as test dependencies, and get rid of the special "precompile" project.

[DilumAluthge]

Why do you need to activate a separate project?

[timholy]

LoopVectorization is used to build LVUser, so LVUser's environment has to be able to find LoopVectorization. And then we load the LVUser package and test whether this caused invalidations.

[DilumAluthge]

Hmmm. Might be cleaner to do this in a subprocess. Let me give this a bit of thought.

[timholy]

what's the best way to make sure it finds the LV we're testing

This version may fix it 🤞 because in the Manifest for LVUser I specify the path for LoopVectorizations as ../../...

[chriselrod]

Great, looks good, thank you!