[PATCH] D69498: IR: Invert convergent attribute handling

[Mehdi AMINI via Phabricator via llvm-commits]

mehdi_amini added a comment.

Talked with Matt at the last social, they thought I was strongly opposed to this: I am not, and I told them I would clarify here:

I don't buy the argument of "frontend projects have consistently run into problems related to this", I think this is not a good justification for this change. But this change seems like it can be motivated without this argument.
I also have been trying to help brainstorming alternatives just to see if we could get something that would fit what Matt is looking to achieve while getting past @rjmccall concerns, not because I am against this change.


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

https://reviews.llvm.org/D69498