RPCS3/llvm-mirror
1
0
Fork 0
mirror of https://github.com/RPCS3/llvm-mirror.git synced 2024-12-30 15:45:26 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Eliminate the explicit volatile fields in LoadInst and StoreInst. This shrinks

Browse Source 
LoadInst from 60 -> 56 bytes and StoreInst from 76 -> 72 bytes.

Note however, that this doesn't actually save any memory on common systems
where 'malloc' returns 8-byte aligned memory, as the saved space is replaced
by useless alignment padding.  :(

llvm-svn: 20046
This commit is contained in:
Chris Lattner 2005-02-05 01:44:18 +00:00
parent ec9411df83
commit 7eb45f89a5
1 changed files with 11 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

23
include/llvm/Instructions.h
View File

@ -167,14 +167,14 @@ public:
// LoadInst Class
//===----------------------------------------------------------------------===//
/// LoadInst - an instruction for reading from memory
/// LoadInst - an instruction for reading from memory. This uses the
/// SubclassData field in Value to store whether or not the load is volatile.
///
class LoadInst : public UnaryInstruction {
bool Volatile; // True if this is a volatile load
LoadInst(const LoadInst &LI)
: UnaryInstruction(LI.getType(), Load, LI.getOperand(0)),
Volatile(LI.isVolatile()) {
: UnaryInstruction(LI.getType(), Load, LI.getOperand(0)) {
setVolatile(LI.isVolatile());
#ifndef NDEBUG
AssertOK();
#endif
@ -191,11 +191,11 @@ public:
/// isVolatile - Return true if this is a load from a volatile memory
/// location.
///
bool isVolatile() const { return Volatile; }
bool isVolatile() const { return SubclassData; }
/// setVolatile - Specify whether this is a volatile load or not.
///
void setVolatile(bool V) { Volatile = V; }
void setVolatile(bool V) { SubclassData = V; }
virtual LoadInst *clone() const;
@ -224,11 +224,10 @@ public:
///
class StoreInst : public Instruction {
Use Ops[2];
bool Volatile; // True if this is a volatile store
StoreInst(const StoreInst &SI) : Instruction(SI.getType(), Store, Ops, 2),
Volatile(SI.isVolatile()) {
StoreInst(const StoreInst &SI) : Instruction(SI.getType(), Store, Ops, 2) {
Ops[0].init(SI.Ops[0], this);
Ops[1].init(SI.Ops[1], this);
setVolatile(SI.isVolatile());
#ifndef NDEBUG
AssertOK();
#endif
@ -245,11 +244,11 @@ public:
/// isVolatile - Return true if this is a load from a volatile memory
/// location.
///
bool isVolatile() const { return Volatile; }
bool isVolatile() const { return SubclassData; }
/// setVolatile - Specify whether this is a volatile load or not.
///
void setVolatile(bool V) { Volatile = V; }
void setVolatile(bool V) { SubclassData = V; }
/// Transparently provide more efficient getOperand methods.
Value *getOperand(unsigned i) const {