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

Anastasia Stulova via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jun 17 04:04:58 PDT 2022


Anastasia added a comment.

I wonder if accepting type alias with identifier `vector` while parsing would be simpler to implement? Then you could just add those type aliases into the internal header. I assume user code is not allowed to redefine those i.e. it would result in undefined behavior or something like for most of C++ standard library features...

It has a drawback of needing to check names for every type alias but it is an invalid program when used in other cases anyway so it should not be a concern for the parsing speed.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D128012



More information about the cfe-commits mailing list