[llvm-commits] Enhance C bindings with volatile load/store

[Yiannis Tsiouris]

On 02/24/12 01:13, Chris Lattner wrote:
> On Feb 23, 2012, at 1:14 PM, Yiannis Tsiouris wrote:
>
>    
>> Hey guys,
>>
>> I attach a (trivial) patch that adds support for volatile load/store
>> through the use of the C bindings. It would be nice if it was committed
>> by someone! :-)
>>
>> If something is wrong/not acceptable please let me know!
>>      
> Hi Yiannis,
>
> What do you think of adding a "getVolatile/setVolatile" style of function to the C API instead?  That seems more general purpose.
>
> -Chris
>    
I attach a solution with "getVolatile/setVolatile" but I'm not sure if 
my handling of LoadInst vs. StoreInst is compatible with the LLVM 
standards. Thus, I apologize if my patch hurts your eyes... Any 
suggestion is very much welcome!


~y.

--
Yiannis Tsiouris
Ph.D. student,
Software Engineering Laboratory,
National Technical University of Athens
WWW: http://www.softlab.ntua.gr/~gtsiour

-------------- next part --------------
A non-text attachment was scrubbed...
Name: VolLoadStore-2.diff
Type: text/x-patch
Size: 1663 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20120224/d6f48493/attachment.bin>