Skip to content

[SYCLomatic][CMake] Fix bug that builtin rules cannot be replaced by user-defined rules#2858

Merged
zhimingwang36 merged 4 commits into
oneapi-src:SYCLomaticfrom
tomflinda:fix_implicit_migration_rule_bug
Jun 5, 2025
Merged

[SYCLomatic][CMake] Fix bug that builtin rules cannot be replaced by user-defined rules#2858
zhimingwang36 merged 4 commits into
oneapi-src:SYCLomaticfrom
tomflinda:fix_implicit_migration_rule_bug

Conversation

@tomflinda

Copy link
Copy Markdown
Contributor

Signed-off-by: chenwei.sun chenwei.sun@intel.com

@tomflinda tomflinda requested a review from a team as a code owner May 27, 2025 08:02
@tomflinda tomflinda requested review from intwanghao and ziranzha May 27, 2025 08:02
@tomflinda tomflinda force-pushed the fix_implicit_migration_rule_bug branch 3 times, most recently from 5212eae to 443d417 Compare May 27, 2025 08:50
Comment thread clang/test/dpct/cmake_migration/case_068/migration_ref.log Outdated
tomflinda added 3 commits June 3, 2025 14:19
… user-definded rules

Signed-off-by: chenwei.sun <chenwei.sun@intel.com>
Signed-off-by: chenwei.sun <chenwei.sun@intel.com>
Signed-off-by: chenwei.sun <chenwei.sun@intel.com>
@tomflinda tomflinda force-pushed the fix_implicit_migration_rule_bug branch from 46deeb2 to 66953dc Compare June 3, 2025 07:05
Comment thread clang/lib/DPCT/MigrateScript/MigrateCmakeScript.cpp Outdated
@tomflinda tomflinda changed the title [SYCLomatic][CMake] Fix bug that implicit rules cannot be replaced by user-defined rules [SYCLomatic][CMake] Fix bug that builtin rules cannot be replaced by user-defined rules Jun 4, 2025
Signed-off-by: chenwei.sun <chenwei.sun@intel.com>
@tomflinda tomflinda force-pushed the fix_implicit_migration_rule_bug branch from f6ab2b9 to 26efd17 Compare June 4, 2025 02:56
@zhimingwang36 zhimingwang36 merged commit 78c271a into oneapi-src:SYCLomatic Jun 5, 2025
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants