intel / intel/llvm

Do we need to add duplicate attributes

Open
#4,119 0 comments 0 reactions 1 assignee Claimed by @smanna12 View on GitHub
bug confirmed
Dominant language
LLVM
Stars
1.5k
Forks
854
Avg merge
3d 17h
Merged PRs (30d)
137

Description

ok Thank you. I didn't notice the `implicit` and was wondering why we have so many nodes. I wonder if we actually need to add duplicate attributes (not just SYCLSimdAttr. I mean all) when we collect and propagate attributes. If we don't, we should probably look into that. Anyway, that is outside the scope of this PR . Thanks again for explaining!

_Originally posted by @elizabethandrews in https://github.com/intel/llvm/pull/4084#discussion_r670862252_

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.