remove always inline attribute from hton/ntoh

[hppritcha]

for ob1 headers when configuring for hetero support.

Signed-off-by: Howard Pritchard [email protected]

[hppritcha]

@jsquyres give this a try

[hppritcha]

related to #9972

[jsquyres]

Yes, removing that "always inline" fixes the compile issue.

Is that a problem, performance-wise, though?

FYI @bwbarrett and @bosilca, who tend to care about changes in OB1.

[bosilca]

I don't think we should have marked this function as always_inline, because according to the documentation this attribute is for functions that would not work properly if not inlined. We should still ask for the function to be inlined if possible, but let the compiler make the decision.

[jsquyres]

I read @bosilca's comment as 👍

[awlauria]

Refs: #9097