https://github.com/bogner commented:

This is looking pretty good, though I have some misgivings about 
`CommonHLSLTargetCodeGenInfo` and think it might be better to introduce a 
`HLSLLayoutBuilder` that can be used by targets as necessary instead. If you 
take that suggestion a couple of my comments become moot.

https://github.com/llvm/llvm-project/pull/124886
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to