[cfe-commits] r69385 - in /cfe/trunk: include/clang/AST/Stmt.h include/clang/Frontend/PCHBitCodes.h include/clang/Frontend/PCHWriter.h lib/AST/Stmt.cpp lib/Frontend/PCHReader.cpp lib/Frontend/PCHWriter.cpp test/PCH/asm.c test/PCH/asm.h

[Douglas Gregor]

On Jan 4, 2010, at 2:15 PM, Mike Stump wrote:

> On Apr 17, 2009, at 1:57 PM, Douglas Gregor wrote:
>> Author: dgregor
>> Date: Fri Apr 17 15:57:14 2009
>> New Revision: 69385
>> 
>> URL: http://llvm.org/viewvc/llvm-project?rev=69385&view=rev
>> Log:
>> PCH support for inline assembly statements.
>> 
>> This completes support for all of C (+ extensions). We can (again)
>> build a PCH file for Carbon.h.
> 
>> --- cfe/trunk/include/clang/AST/Stmt.h (original)
>> +++ cfe/trunk/include/clang/AST/Stmt.h Fri Apr 17 15:57:14 2009
> 
>>  bool isVolatile() const { return IsVolatile; }
>> +  void setVolatile(bool V) { IsVolatile = V; }
>>  bool isSimple() const { return IsSimple; }
>> +  void setSimple(bool V) { IsSimple = false; }
> 
> Shouldn't that be:
> 
>   void setSimple(bool V) { IsSimple = V; }
> 
> ?

Yikes, thanks!

	- Doug