Add docs for function attributes hot/cold

[OfekShilon]

Summary

Following this, add docs for the hot/cold function attributes

[aaron.ballman]

Thank you for adding the docs! I'm not well-versed enough in hot/cold behavior and PGO to comment on the documentation accuracy, so I'm hopeful someone can sign off on that bit.

clang/include/clang/Basic/Attr.td

1077

I think the change on this line was unintentional.

1523

I think the change on this line was unintentional.

clang/include/clang/Basic/AttrDocs.td

5248

5255–5256

[davidxl]

LGTM from the content point of view. Please also address aaron's comment.

[OfekShilon]

Addressed Aaron's comments (thanks!). Hope I got the update-diff procedure right.
I don't have commit permissions, would appreciate if someone could push this.

[aaron.ballman]

In D130933#3710682, @OfekShilon wrote:

Addressed Aaron's comments (thanks!). Hope I got the update-diff procedure right.

It looks like something might have gone wrong because the diff looks like it's now a diff against something other than the main branch (it looks like it's adding the SimpleHandler lines in Attr.td, but those lines already exist in main).

I don't have commit permissions, would appreciate if someone could push this.

Once we get the patch fixed up, I'm happy to land on your behalf. What name and email address would you like me to use for patch attribution?

[OfekShilon]

This comment was removed by OfekShilon.

[aaron.ballman]

I'm not a git expert by any means, but you might be able to address the patch issues with git diff -U999 main > foo.patch from the root llvm-project directory so that you're getting a diff with 999 lines of context and the differences are against the main branch.

[OfekShilon]

My apologies for the mess, hopefully the diff I just uploaded includes all the right fixes.

[aaron.ballman]

In D130933#3713277, @OfekShilon wrote:

My apologies for the mess, hopefully the diff I just uploaded includes all the right fixes.

No worries! If I had a nickel for every time I've been confused by git... I'd have a lot of nickels.

This version LGTM! I can land it for you, but what name and email address would you like me to use for patch attribution?

[OfekShilon]

@aaron.ballman 'Ofek Shilon', [e-mail] is fine.
Thanks!

[aaron.ballman]

In D130933#3713437, @OfekShilon wrote:

@aaron.ballman 'Ofek Shilon', [e-mail] is fine.
Thanks!

Thank you! I've landed on your behalf.

clang/include/clang/Basic/Attr.td

   let Spellings = [GCC<"cold">];
   let Subjects = SubjectList<[Function]>;
   let Documentation = [ColdFunctionEntryDocs];
+  let SimpleHandler = 1;
 }
 
 def Common : InheritableAttr {

   let Spellings = [GCC<"hot">];
   let Subjects = SubjectList<[Function]>;
   let Documentation = [HotFunctionEntryDocs];
+  let SimpleHandler = 1;
 }
 def : MutualExclusions<[Hot, Cold]>;
 

clang/include/clang/Basic/AttrDocs.td

   let Category = DocCatFunction;
   let Content = [{
 ``__attribute__((hot))`` marks a function as hot, as a manual alternative to PGO hotness data. 
-In case PGO data is available too user annotated ``__attribute__((hot))`` overwrites profile count based hotness (unlike ``__attribute__((cold))``).
+If PGO data is available, the annotation ``__attribute__((hot))`` overrides the profile count based hotness (unlike ``__attribute__((cold))``).
 }];
 }
 

   let Category = DocCatFunction;
   let Content = [{
 ``__attribute__((cold))`` marks a function as cold, as a manual alternative to PGO hotness data. 
-In case PGO data is available too profile count based hotness overwrites user annotated ``__attribute__((cold))`` (unlike ``__attribute__((hot))``).
+If PGO data is available, the profile count based hotness overrides the ``__attribute__((cold))`` annotation (unlike ``__attribute__((hot))``).
 }];
 }