[PATCH] D90275: [clang][IR] Add support for leaf attribute

Nuno Lopes via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Aug 10 08:41:40 PDT 2022


nlopes added a comment.

In D90275#3707793 <https://reviews.llvm.org/D90275#3707793>, @nlopes wrote:

> In D90275#3671983 <https://reviews.llvm.org/D90275#3671983>, @gulfem wrote:
>
>> In D90275#3671019 <https://reviews.llvm.org/D90275#3671019>, @nlopes wrote:
>>
>>> Could you please add a description of this attribute to LangRef?
>>> We need the semantics of this.
>>>
>>> Thank you!
>>
>> Sure, I'll do that!
>
> ping?

Since the patch author has been ignoring my ask (and is active), and since documenting IR features is of paramount importance, I suggest we revert this patch. I think 3 weeks to document something is sufficient, and the author didn't provide an ETA.
I would like to ask everyone to reject all future patches that change the IR and don't include documentation, as experience says that we'll get miscompilations due to different people having different ideas of the semantics.

TL;DR: I'll revert this patch on Friday if no one chimes in. Thank you!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D90275



More information about the cfe-commits mailing list