Skip to content

Fix OpenCLOptTable build: port to LLVM's rewritten OptTable API - #887

Merged
wenju-he merged 1 commit into
intel:mainfrom
wenju-he:fix-build-OptTable
Sep 20, 2026
Merged

wenju-he merged 1 commit into
intel:mainfrom
wenju-he:fix-build-OptTable

Conversation

@wenju-he

Copy link
Copy Markdown
Contributor

LLVM commit 40a6a441e7a9 ("[Option] Emit the Info table from TableGen. NFC", #224508) removed GenericOptTable and the OPTTABLE_STR_TABLE_CODE/ OPTTABLE_PREFIXES_TABLE_CODE generator macros in favor of a single OptTable::Tables struct produced by OPTTABLE_CODE. Update OpenCLOptTable and OpenCLCompileOptTable to construct from optionTables() instead of a hand-built OptTable::Info array.

Fix https://github.com/intel/opencl-clang/actions/runs/35477853263/job/105990104610

LLVM commit 40a6a441e7a9 ("[Option] Emit the Info table from TableGen.
NFC", #224508) removed GenericOptTable and the OPTTABLE_STR_TABLE_CODE/
OPTTABLE_PREFIXES_TABLE_CODE generator macros in favor of a single
OptTable::Tables struct produced by OPTTABLE_CODE. Update OpenCLOptTable
and OpenCLCompileOptTable to construct from optionTables() instead of a
hand-built OptTable::Info array.

Fix https://github.com/intel/opencl-clang/actions/runs/35477853263/job/105990104610

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@wenju-he
wenju-he enabled auto-merge (squash) September 20, 2026 02:50
@wenju-he
wenju-he requested a review from cdai2 September 20, 2026 02:50

@cdai2 cdai2 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@wenju-he
wenju-he merged commit 8aa9bf2 into intel:main Sep 20, 2026
2 of 4 checks passed
@wenju-he
wenju-he deleted the fix-build-OptTable branch September 20, 2026 22:39
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.

2 participants