Skip to content
Snippets Groups Projects

[Fix] Vectorisation of Abs

Merged Markus Holzer requested to merge holzer/pystencils:VectorAbs into master
All threads resolved!

When an abs function is encountered the Vector printer checks if its argument is a 'VectorMemoryAccess'. This causes two issues 1. only abs functions containing a field access can be vectorised and 2. even worse, if the abs function does not contain a 'VectorMemoryAccess' the scalar version is printed which causes faulty code.

This MR simply removes the condition.

Merge request reports

Checking approval status
Test summary results are loading

Merged by Michael KuronMichael Kuron 6 months ago (Sep 28, 2024 10:20am UTC)

Loading

Pipeline #69291 failed

Pipeline failed for d8586152 on master

Test coverage 87.07% from 1 job

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply