[PATCH] D128012: [HLSL] Add ExternalSemaSource & vector alias

Tue Jun 21 02:15:35 PDT 2022

Anastasia added a comment.

In D128012#3592126 <https://reviews.llvm.org/D128012#3592126>, @beanz wrote:

> In D128012#3592098 <https://reviews.llvm.org/D128012#3592098>, @Anastasia wrote:
>
>> aha, are you having a lot of these types then? Ok that sounds similar problem to OpenCL builtin functions as we had to go away from a simple header include due to long parsing time.
>
> Very similar problem. We have a two-part problem where we both have enough of them that parsing a header would be too slow _and_ the HLSL language actually can't represent them. The later we're working to fix. My hope is that eventually we can write the HLSL libraries in HLSL and use a pre-compilation step (PCH or Module-style) to get past the performance issues.

PCH have large in-memory size compared to the sources, but if it's not an issue then it should be reasonable.

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D128012/new/

https://reviews.llvm.org/D128012