[llvm-commits] [llvm] r79498 - /llvm/trunk/docs/ProgrammersManual.html

[John McCall]

Eli Friedman wrote:
> On Wed, Aug 19, 2009 at 5:29 PM, Chris Lattner<sabre at nondot.org> wrote:
>   
>> @@ -2422,7 +2422,7 @@
>>  <p>
>>  Conceptually, <tt>LLVMContext</tt> provides isolation.  Every LLVM entity
>>  (<tt>Module</tt>s, <tt>Value</tt>s, <tt>Type</tt>s, <tt>Constant</tt>s, etc.)
>> -in LLVM's in-memory IR belongs to an <tt>LLVMContext</tt>.  Entities in
>> +in LLVM's in-memory IR belong to an <tt>LLVMContext</tt>.  Entities in
>>     
>
> I don't mean to be pedantic, but I think "belongs" is right here:
> "entity" is singular, even though the sentence is referring to a set
> of entities.
>   

I agree.  That said, the old wording jarred with the parenthetical 
list;  the better fix would be
    s/Every LLVM entity/All entities/

John.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20090819/b8b0e28b/attachment.html>