(Towards #3060) reference accesses for intrinsicCall

[LonelyCat124]

No description provided.

[LonelyCat124]

Follow-on to #3119

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 99.95%. Comparing base (c9a794c) to head (aedfeb6).
⚠️ Report is 58 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #3125      +/-   ##
==========================================
- Coverage   99.95%   99.95%   -0.01%     
==========================================
  Files         382      382              
  Lines       54107    54049      -58     
==========================================
- Hits        54085    54027      -58     
  Misses         22       22              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[LonelyCat124]

@sergisiso I have a first implementation of this now if you want to examine it if you have time. Its currently "blocked" by #3119 (which is in turn dependent on #3110 ) so there's very much no rush on this, especially if I do solve some other issues before we merge 3319.

[LonelyCat124]

@sergisiso Ready for a review when you're back from leave.

[sergisiso]

Almost there now @LonelyCat124, just one more possible simplification and small clean ups.

[LonelyCat124]

@sergisiso Ready for another look, I did swap to the default as you suggested.

[sergisiso]

All looks good now @LonelyCat124, it ITs come green I will proceed to merge.