Re: [Linux-nvdimm] [PATCH 4/6] SQUSHME: pmem: Micro cleaning

From: Matthew Wilcox
Date: Tue Mar 31 2015 - 15:40:24 EST


On Tue, Mar 31, 2015 at 06:43:40PM +0300, Boaz Harrosh wrote:
> But not here. This is not an optimization this is the *error path*.
> What I'm saying is:
> "No compiler nor CPU, even if 99% of the time this branch is taken
> I still consider it cold. Because it is the error case and
> I do not care for it"

GCC already understands that "if (foo) goto FORWARD_LABEL" should be
predicted unlikely by default. All you're doing is cluttering the
source code.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/