Comment on Comments

OK:

if (vars.empty()) { 
  // Store null env pointer if environment is empty 
  builder.CreateStore(
      llvm::ConstantPointerNull::getNullValue( 
          clo_env_slot->getType()
             ->getContainedType(0)), 
      clo_env_slot, 
      /* isVolatile= */ false);
}

 

Better:

if (vars.empty()) {
  storeNullPointerToSlot(clo_env_slot);
}
Advertisements

Leave a Reply

Fill in your details below or click an icon to log in:

Gravatar
WordPress.com Logo

You are commenting using your WordPress.com account. ( Log Out /  Change )

Google+ photo

You are commenting using your Google+ account. ( Log Out /  Change )

Twitter picture

You are commenting using your Twitter account. ( Log Out /  Change )

Facebook photo

You are commenting using your Facebook account. ( Log Out /  Change )

Cancel

Connecting to %s