On Thu, 9 May 2019 09:51:59 -0700 Linus Torvalds <torva...@linux-foundation.org> wrote:
> On Thu, May 9, 2019 at 6:01 AM Steven Rostedt <rost...@goodmis.org> wrote: > > > > This patch works. Can I get your Signed-off-by for it? > > Yes. Please write some kind of comprehensible commit log for it, but How's this: "Peter Zijlstra noticed that with CONFIG_PROFILE_ALL_BRANCHES, the "if" macro converts the conditional to an array index. This can cause GCC to create horrible code. When there are nested ifs, the generated code uses register values to encode branching decisions. Josh Poimboeuf found that replacing the define "if" macro from using the condition as an array index and incrementing the branch statics with an if statement itself, reduced the asm complexity and shrinks the generated code quite a bit. But this can be simplified even further by replacing the internal if statement with a ternary operator. Reported-by: Peter Zijlstra (Intel) <pet...@infradead.org> Reported-by: Josh Poimboeuf <jpoim...@redhat.com> " > > Signed-off-by: Linus Torvalds <torva...@linux-foundation.org> Thanks, -- Steve > > for the patch itself. >
