 Chromium Code Reviews
 Chromium Code Reviews Issue 515993002:
  Align function starts to target-specific bundle alignment.  (Closed) 
  Base URL: https://chromium.googlesource.com/native_client/pnacl-subzero.git@master
    
  
    Issue 515993002:
  Align function starts to target-specific bundle alignment.  (Closed) 
  Base URL: https://chromium.googlesource.com/native_client/pnacl-subzero.git@master| Index: src/IceTargetLowering.h | 
| diff --git a/src/IceTargetLowering.h b/src/IceTargetLowering.h | 
| index 6bf142487eb520aaee58570acef2341a84fbf0ed..c3e468da192acc09cc8469d3405bf6c35d8d07bd 100644 | 
| --- a/src/IceTargetLowering.h | 
| +++ b/src/IceTargetLowering.h | 
| @@ -137,6 +137,7 @@ public: | 
| virtual bool hasFramePointer() const { return false; } | 
| virtual SizeT getFrameOrStackReg() const = 0; | 
| virtual size_t typeWidthInBytesOnStack(Type Ty) const = 0; | 
| + virtual SizeT getBundleP2Align() const = 0; | 
| 
Jim Stichnoth
2014/08/28 20:26:12
Rename this something like getBundleP2AlignLog2Byt
 
jvoung (off chromium)
2014/08/28 21:02:22
Renamed to getBundleAlignLog2Bytes() -- P2 was mea
 
Jim Stichnoth
2014/08/28 21:22:21
Ah, I didn't realize what the p2 stood for. :)  In
 
jvoung (off chromium)
2014/08/28 22:38:13
Well, Log2 is probably more obvious =)
 | 
| bool hasComputedFrame() const { return HasComputedFrame; } | 
| bool shouldDoNopInsertion() const; | 
| int32_t getStackAdjustment() const { return StackAdjustment; } |