[PATCH] D70365: intrinsics attribute opt-out list proposal.

[Stefan Stipanovic via Phabricator via llvm-commits]

sstefan1 added a comment.

In D70365#2128556 <https://reviews.llvm.org/D70365#2128556>, @jdoerfert wrote:

> In D70365#2123539 <https://reviews.llvm.org/D70365#2123539>, @efriedma wrote:
>
> >
>
>
> @sstefan1 could u update the patch? We then send another email out with the list of 5 attributes we will make opt-out for intrinsics. A week or so later, assuming no one objected too much, we'll land it. In the meantime we need to update the target independent ones to opt-out appropriately. I guess my nosync list will catch almost all of them already but we'll have to give it another go as the patch is updated. WDYT?


I was under the impression that the attributes will be made default in another patch. Right now we don't opt out of the default attributes, but none are set to be default yet. 
If it is better to do this in this patch, no problem. I can send an email, on Monday maybe?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D70365