[PATCH] D134067: [HLSL] Enable availability attribute
Erich Keane via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Sep 16 13:33:29 PDT 2022
erichkeane added a comment.
Still looks good to me, no nits. As far as using the bookmarks, I tend to use them for things that are 'far away' instead of 'right here', and stick to the @+/@- bits for 'local' ones.
================
Comment at: clang/test/SemaHLSL/AvailabilityMarkup.hlsl:13
+void fn() {
+ // expected-warning@#fn6_0_site {{'fn6_0' is only available on HLSL ShaderModel 6.0 or newer}}
+ // expected-note@#fn6_0 {{'fn6_0' has been marked as being introduced in HLSL ShaderModel 6.0 here, but the deployment target is HLSL ShaderModel 5.0}}
----------------
Your preference here, but the `@+3` here is acceptable when they are 'local'. I have no preference either way.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D134067/new/
https://reviews.llvm.org/D134067
More information about the cfe-commits
mailing list