Mirrors/servo
1
0
Fork 0
mirror of https://github.com/servo/servo.git synced 2025-08-02 20:20:14 +01:00
Code Issues Projects Releases Packages Wiki Activity

Convert inline(always) to inline in CodegenRust and jstraceable.

Browse source 
This results in a 14% compile time improvement.

See https://gist.github.com/brson/b48dd03b06c406be68e6
This commit is contained in:
Brian Anderson 2015-06-05 14:56:11 -07:00
parent 68d977c5f0
commit 172fbcad27
2 changed files with 11 additions and 11 deletions
Download patch file Download diff file Expand all files Collapse all files

2
components/plugins/jstraceable.rs
View file

@ -50,7 +50,7 @@ pub fn expand_jstraceable(cx: &mut ExtCtxt, span: Span, mitem: &MetaItem, item:
args: vec!(ty::Ptr(box ty::Literal(ty::Path::new(vec!("js","jsapi","JSTracer"))), args: vec!(ty::Ptr(box ty::Literal(ty::Path::new(vec!("js","jsapi","JSTracer"))),
ty::Raw(ast::MutMutable))), ty::Raw(ast::MutMutable))),
ret_ty: ty::nil_ty(), ret_ty: ty::nil_ty(),
attributes: vec![quote_attr!(cx, #[inline(always)])], attributes: vec![quote_attr!(cx, #[inline])],
is_unsafe: false, is_unsafe: false,
combine_substructure: combine_substructure(box jstraceable_substructure) combine_substructure: combine_substructure(box jstraceable_substructure)
} }

20
components/script/dom/bindings/codegen/CodegenRust.py
View file

@ -1887,7 +1887,7 @@ class CGAbstractMethod(CGThing):
def _decorators(self): def _decorators(self):
decorators = [] decorators = []
if self.alwaysInline: if self.alwaysInline:
decorators.append('#[inline(always)]') decorators.append('#[inline]')
if self.extern: if self.extern:
decorators.append('unsafe') decorators.append('unsafe')
@ -5478,7 +5478,7 @@ impl ${selfName} for ${baseName} {
cast = [CGGeneric(string.Template("""\ cast = [CGGeneric(string.Template("""\
pub struct ${name}Cast; pub struct ${name}Cast;
impl ${name}Cast { impl ${name}Cast {
#[inline(always)] #[inline]
pub fn to_ref<'a, T: ${toBound}+Reflectable>(base: JSRef<'a, T>) -> Option<JSRef<'a, ${name}>> { pub fn to_ref<'a, T: ${toBound}+Reflectable>(base: JSRef<'a, T>) -> Option<JSRef<'a, ${name}>> {
match base.${checkFn}() { match base.${checkFn}() {
true => Some(unsafe { mem::transmute(base) }), true => Some(unsafe { mem::transmute(base) }),
@ -5486,7 +5486,7 @@ impl ${name}Cast {
} }
} }
#[inline(always)] #[inline]
pub fn to_borrowed_ref<'a, 'b, T: ${toBound}+Reflectable>(base: &'a JSRef<'b, T>) -> Option<&'a JSRef<'b, ${name}>> { pub fn to_borrowed_ref<'a, 'b, T: ${toBound}+Reflectable>(base: &'a JSRef<'b, T>) -> Option<&'a JSRef<'b, ${name}>> {
match base.${checkFn}() { match base.${checkFn}() {
true => Some(unsafe { mem::transmute(base) }), true => Some(unsafe { mem::transmute(base) }),
@ -5494,7 +5494,7 @@ impl ${name}Cast {
} }
} }
#[inline(always)] #[inline]
#[allow(unrooted_must_root)] #[allow(unrooted_must_root)]
pub fn to_layout_js<T: ${toBound}+Reflectable>(base: &LayoutJS<T>) -> Option<LayoutJS<${name}>> { pub fn to_layout_js<T: ${toBound}+Reflectable>(base: &LayoutJS<T>) -> Option<LayoutJS<${name}>> {
unsafe { unsafe {
@ -5505,7 +5505,7 @@ impl ${name}Cast {
} }
} }
#[inline(always)] #[inline]
pub fn to_temporary<T: ${toBound}+Reflectable>(base: Temporary<T>) -> Option<Temporary<${name}>> { pub fn to_temporary<T: ${toBound}+Reflectable>(base: Temporary<T>) -> Option<Temporary<${name}>> {
match base.root().r().${checkFn}() { match base.root().r().${checkFn}() {
true => Some(unsafe { mem::transmute(base) }), true => Some(unsafe { mem::transmute(base) }),
@ -5513,28 +5513,28 @@ impl ${name}Cast {
} }
} }
#[inline(always)] #[inline]
pub fn from_ref<'a, T: ${fromBound}+Reflectable>(derived: JSRef<'a, T>) -> JSRef<'a, ${name}> { pub fn from_ref<'a, T: ${fromBound}+Reflectable>(derived: JSRef<'a, T>) -> JSRef<'a, ${name}> {
unsafe { mem::transmute(derived) } unsafe { mem::transmute(derived) }
} }
#[inline(always)] #[inline]
pub fn from_borrowed_ref<'a, 'b, T: ${fromBound}+Reflectable>(derived: &'a JSRef<'b, T>) -> &'a JSRef<'b, ${name}> { pub fn from_borrowed_ref<'a, 'b, T: ${fromBound}+Reflectable>(derived: &'a JSRef<'b, T>) -> &'a JSRef<'b, ${name}> {
unsafe { mem::transmute(derived) } unsafe { mem::transmute(derived) }
} }
#[inline(always)] #[inline]
#[allow(unrooted_must_root)] #[allow(unrooted_must_root)]
pub fn from_layout_js<T: ${fromBound}+Reflectable>(derived: &LayoutJS<T>) -> LayoutJS<${name}> { pub fn from_layout_js<T: ${fromBound}+Reflectable>(derived: &LayoutJS<T>) -> LayoutJS<${name}> {
unsafe { mem::transmute_copy(derived) } unsafe { mem::transmute_copy(derived) }
} }
#[inline(always)] #[inline]
pub fn from_temporary<T: ${fromBound}+Reflectable>(derived: Temporary<T>) -> Temporary<${name}> { pub fn from_temporary<T: ${fromBound}+Reflectable>(derived: Temporary<T>) -> Temporary<${name}> {
unsafe { mem::transmute(derived) } unsafe { mem::transmute(derived) }
} }
#[inline(always)] #[inline]
pub fn from_actual<'a, T: ${fromBound}+Reflectable>(derived: &'a T) -> &'a ${name} { pub fn from_actual<'a, T: ${fromBound}+Reflectable>(derived: &'a T) -> &'a ${name} {
unsafe { mem::transmute(derived) } unsafe { mem::transmute(derived) }
} }