Mirror of forum.nim-lang.org

3277 :: real inline for inline procs or converters

• adrianv (orginal) [2017-10-24T14:40:48+02:00] view original

  Hi,

  just wanted to know if there is a reason (other than we more important things to do than that), why procs and converters are not inlined by nim when marked as such. I know you can always use a template - but not for converters (normally they are just a cast to make the typing the way you want). For js this could be an improvement too.

---

• Stefan_Salewski (orginal) [2017-10-24T15:35:19+02:00] view original

  What is exactly your problem? Can you give examples?

  The real inlining is of course done by the C compiler, maybe your one is too weak. And using Link Time Optimization LTO can give even better inlining. GCC and clang support -flto flag, where clang needed gold linker for that in the past.

---

• adrianv (orginal) [2017-10-24T16:46:42+02:00] view original

  not a real problem - but for JavaScript there is no inlining of procs. And for converters I would prefer to have the choice if they are inlined by nim or not, but the more I think about it, the benefit would be just cosmetic.

---

• Araq (orginal) [2017-10-25T16:18:39+02:00] view original

  The lack of inlining for JS can be a problem but it's also really hard to tell whether the JIT did inline or not.

---

• Varriount (orginal) [2017-10-25T16:20:16+02:00] view original

  You can always run your output code through Google Closure Compiler.

---

• jxy (orginal) [2017-10-27T19:45:49+02:00] view original

  Try my just uploaded module, metatools. There is a macro, inlineProcs. See one of the test examples: https://github.com/jxy/metatools/blob/master/test/tinline000.nim

  I'll put in documents later.

---

• Udiknedormin (orginal) [2017-11-04T23:18:22+01:00] view original

  @jxy Good to know it was already implemented as I was actually planning to implement it myself. ^^" I haven't tried your lib in practice but the code looks pretty nice.

---

• jxy (orginal) [2017-11-09T21:46:28+01:00] view original

  @Udiknedormin, I'm trying to split out some reusable code from my projects in the hope that other people might find it useful. Let me know if it helps you in any way.

---

• Udiknedormin (orginal) [2017-11-10T08:45:38+01:00] view original

  @jxy Well, I wanted to make a call optimization library, actually. ;) Including various recursion optimizations (I know there is memo but it provides only one opt method and even my pull request to it was never accepted). Inlining is, of course, one of the possible optimizations (so I guess it's a dependency here? Too bad nimble doesn't handle them automagically).

---