[LLVMdev] [RFC] Stackmap and Patchpoint Intrinsic Proposal

[Andrew Trick]

On Oct 18, 2013, at 12:39 PM, Chris Lattner <clattner at apple.com> wrote:

> On Oct 18, 2013, at 11:08 AM, Andrew Trick <atrick at apple.com> wrote:
> 
>>> 
>>> I recommend, this being the case, to replace 'webkit' with 'experimental'. Having webkit in the name implies some dependence on webkit, and there is none. Plus, this functionality will be used by outside projects as soon as it lands in trunk, and I suspect that having webkit in the initial name will end up as a naming incongruity that no one will really think is worth the effort to change.
>> 
>> You’re correct that there is no dependence. I’m fine dropping the webkit name, but only if we can go straight to the final name (no need for “experimental”).
> 
> I think that Hal's idea of "experimental" is the right approach here.  The major thing we want is to avoid having to be backwards compatible with this intrinsic in subsequent llvm releases.  "experimental" sends that message, where webkit does not (and is also bad for the reasons Hal mentions).

Done. I’ll update the patches on llvm-commits.

For the record, I wasn’t aware of any precedent for “llvm.experimental”, but if it will help avoid backward compatibility issues then it’s a good thing.

-Andy

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20131018/009e7080/attachment.html>