[PATCH] D86233: [LangRef] Define maynotprogress attribute

Hal Finkel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 4 17:26:43 PDT 2020


hfinkel added a comment.

In D86233#2256909 <https://reviews.llvm.org/D86233#2256909>, @hfinkel wrote:

> In D86233#2256908 <https://reviews.llvm.org/D86233#2256908>, @nikic wrote:
>
>> Or `might_not_progress` while we're at it :) No requirement to squash the whitespace.
>
> True. Although we squash the whitespace for all other attributes. I don't really like it, and would prefer the underscores, but it would be different.

I take that back. We have null_pointer_is_valid, sanitize_address, etc. now.  I also vote for the underscores.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D86233



More information about the llvm-commits mailing list