inline würde ich Dir generell raten wegzulassen. Das macht der Compiler oder eben nicht. Das Schlüsselwort ist ziemlich überflüssig geworden.
Das stimmt. Inline selbst hinzuschreiben weil man sich bessere Performance erhofft ist ziemlich überflüssig. Außerdem kann man das selbst meistens schlechter beurteilen als der Compiler.
Inline macht aber auch aus anderen Gründen Sinn: Wenn man ein Framework schreibt und eine Funktion dort als inline markiert, taucht sie im Binary nur auf wenn sie verwendet wird. Bei großen Funktionen und engen Größenvorgaben fürs Binary kann man das brauchen (embedded oder demo-bereich).
Inline hat aber auch Nachteile, auch wenn der Compiler das automatisch setzt. Deshalb kann man den Compiler anweisen, es nicht zu benutzen. Wenn das Programm abstürzt oder man es mit dem Debugger durchstept erzeugt eine mit inline markierte Funktion keinen Eintrag in der Stacktrace. Dessen sollte man sich bewusst sein, wenn man auf der Ebene debuggt.