Skip to content

[HIPIFY][fix] LLVM upstream preparations - Part 4 - namespace usage revision#2467

Draft
ranapratap55 wants to merge 1 commit intoROCm:amd-developfrom
ranapratap55:users/ranapratap55/llvm-prep-namespace
Draft

[HIPIFY][fix] LLVM upstream preparations - Part 4 - namespace usage revision#2467
ranapratap55 wants to merge 1 commit intoROCm:amd-developfrom
ranapratap55:users/ranapratap55/llvm-prep-namespace

Conversation

@ranapratap55
Copy link
Copy Markdown
Collaborator

Addresses using namespace violations in headers and main source(.cpp) files to match with LLVM coding standards.

  • Removed using namespace llvm;, using namespace llvm; and used llvm::, std:: in header files.
  • Added using namespace llvm; in main source(.cpp) files

@ranapratap55 ranapratap55 self-assigned this Mar 30, 2026
@ranapratap55 ranapratap55 added enhancement Enhancement fix It fixes bug clang clang compiler related issue or change compatibility Compatibility change or request labels Mar 30, 2026
@ranapratap55 ranapratap55 force-pushed the users/ranapratap55/llvm-prep-namespace branch from d1dc065 to ef050ff Compare March 30, 2026 08:31
@ranapratap55 ranapratap55 changed the title [HIPIFY][fix] LLVM upstream preparations - Part 4 - namespace usage revision [HIPIFY][fix] LLVM upstream preparations - Part 4 - namespace usage revision Mar 30, 2026
@ranapratap55 ranapratap55 marked this pull request as draft March 31, 2026 05:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

clang clang compiler related issue or change compatibility Compatibility change or request enhancement Enhancement fix It fixes bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant