Skip to content
Snippets Groups Projects

flatten localToGlobal in all cases and also for opt builds

We were flattening in one case (perigeeSurface) but in my understanding only for debug builds (special trick from @ssnyder compiling the code with opt even in debug build and using attribute flatten).

The issue is "opt" does not "flatten" (actually the aim of the flatten is to inline the Eigen code)
either with gcc11...

Screenshot_2022-05-17_at_17.34.07

Before this MR in opt

Screenshot_2022-05-17_at_19.08.30 Screenshot_2022-05-17_at_19.09.20 Screenshot_2022-05-17_at_19.09.11

After the MR in opt

Screenshot_2022-05-17_at_19.08.43 Screenshot_2022-05-17_at_19.09.33 Screenshot_2022-05-17_at_19.08.52

ping @ssnyder so as to take a look

For completeness I noticed this here first so have an example handy

So this "inlining" is something that clang does (also previous versions) but gcc stopped doing...
"This" -> is inline certain 5x5 matrix operations it seems .

Edited by Christos Anastopoulos

Merge request reports

Pipeline #3984922 passed

Pipeline passed for f6ae6b3e on ATLAS-EGamma:Surface_flatten_out_of_line_Eigen

Approval is optional

Merged by Vakhtang TsulaiaVakhtang Tsulaia 2 years ago (May 19, 2022 4:26pm UTC)

Merge details

  • Changes merged into master with 75b00db1.
  • Deleted the source branch.

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
Loading