This is an archive of the discontinued LLVM Phabricator instance.

Move HLSL builtins into hlsl namespace
ClosedPublic

Authored by beanz on Oct 14 2022, 10:24 AM.

Details

Summary

Should have done this from the start. Since all the injected AST types
are in the hlsl namespace we should also put the header-defined types
and functions in there too.

This updates the basic_types test to run once with the namespaced types
and once without, and adds using declarations or namespaces calls in
other tests.

Diff Detail

Event Timeline

beanz created this revision.Oct 14 2022, 10:24 AM
Herald added a project: Restricted Project. · View Herald TranscriptOct 14 2022, 10:24 AM
Herald added a subscriber: Anastasia. · View Herald Transcript
beanz requested review of this revision.Oct 14 2022, 10:24 AM
Herald added a project: Restricted Project. · View Herald TranscriptOct 14 2022, 10:24 AM
python3kgae accepted this revision.Oct 15 2022, 10:19 AM
This revision is now accepted and ready to land.Oct 15 2022, 10:19 AM
This revision was automatically updated to reflect the committed changes.