[PATCH] D45534: Add isl operator overloads for isl::pw_aff (Try II)

[Michael Kruse via Phabricator via llvm-commits]

Meinersbur added a comment.

In https://reviews.llvm.org/D45534#1064800, @philip.pfaffe wrote:

> Since you expect users to include the header, the operators should also be declared static.


This is not required. If the inline-function has external linkage, the compiler/linker may (and even must) reuse the same symbol for all cases where the function is not inlined (which is possible at the compiler's judgment). With static, the same function symbol can appear multiple times in the final executable, resulting in larger executables. This is how I understand http://en.cppreference.com/w/cpp/language/inline


Repository:
  rPLO Polly

https://reviews.llvm.org/D45534