bryanpkc added inline comments.

================
Comment at: clang/lib/Headers/CMakeLists.txt:332
+  # Generate arm_sme.h
+  clang_generate_header(-gen-arm-sme-header arm_sme.td arm_sme.h)
   # Generate arm_bf16.h
----------------
sdesmalen wrote:
> The ACLE specification is still in a draft (ALP) state, which means there may 
> still be subject to significant changes. To avoid users from using this 
> implementation with the expectation that their code is compliant going 
> forward, it would be good to rename the header file to something that makes 
> it very clear this feature is not yet ready to use. I'm thinking of something 
> like `arm_sme_draft_spec_subject_to_change.h`. When the specification goes 
> out of draft, we can rename it to `arm_sme.h`. Could you rename the file for 
> now?
Would something shorter like `arm_sme_draft.h` or `arm_sme_experimental.h` 
suffice?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D127910

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to