[libcxx-commits] [PATCH] D119765: [libc++][AIX] Remove "pragma priority" from locale.cpp

[Xing Xue via Phabricator via libcxx-commits]

xingxue added inline comments.


================
Comment at: libcxx/src/locale.cpp:132
-// Set priority to INT_MIN + 256 + 150
-# pragma priority ( -2147483242 )
-#endif
----------------
hubert.reinterpretcast wrote:
> > The `pragma priority` guarded for AIX in locale.cpp is no longer useful and is ignored by the current AIX build compilers.  This patch removes it from the source.
> 
> @xingxue, this is no longer useful because you observed that compiling the file results in no functions being generated to perform non-local initialization?
Right, there are no static constructor functions (__sinit*()) in the resulting locale.cpp.o file.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D119765/new/

https://reviews.llvm.org/D119765