Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[LLVM][L1] Dead assignment in L1NNCaloTauEmulator::produce #46265

Open
iarspider opened this issue Oct 4, 2024 · 4 comments
Open

[LLVM][L1] Dead assignment in L1NNCaloTauEmulator::produce #46265

iarspider opened this issue Oct 4, 2024 · 4 comments

Comments

@iarspider
Copy link
Contributor

LLVM analyzer reports dead assignment in L1NNCaloTauEmulator::produce: link

 float idScore = -1.;
 if (!VsPuId.method().empty()) {
   idScore = VsPuId.passID(*cl3dIt, cluster);
   idScore = cluster.egVsPUMVAOut();
 }

@cms-sw/l1-l2 could you please check if the return value of VsPuId.passID should be discarded or used?

@iarspider
Copy link
Contributor Author

assign l1

@cmsbuild
Copy link
Contributor

cmsbuild commented Oct 4, 2024

New categories assigned: l1

@aloeliger,@epalencia you have been requested to review this Pull request/Issue and eventually sign? Thanks

@cmsbuild
Copy link
Contributor

cmsbuild commented Oct 4, 2024

cms-bot internal usage

@cmsbuild
Copy link
Contributor

cmsbuild commented Oct 4, 2024

A new Issue was created by @iarspider.

@Dr15Jones, @antoniovilela, @makortel, @mandrenguyen, @rappoccio, @sextonkennedy, @smuzaffar can you please review it and eventually sign/assign? Thanks.

cms-bot commands are listed here

@iarspider iarspider changed the title [LLVM] Dead assignment in L1NNCaloTauEmulator::produce [LLVM][L1] Dead assignment in L1NNCaloTauEmulator::produce Oct 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants