[PATCH] D103615: [Clang] Add option for vector compare compatibility.

[Nemanja Ivanovic via Phabricator via cfe-commits]

nemanjai added a comment.

I haven't had time to review this yet, but I just wanted to chime in on the option spelling and description. I think we should go with:

  -faltivec-src-compat={xl|gcc|mixed}
    Source-level compatibility for Altivec vectors (for PowerPC targets). This
    includes results of vector comparison (scalar for 'xl', vector for 'gcc')
    as well as behavior when initializing with a scalar (splatting for 'xl',
    element zero only for 'gcc'). For 'mixed', the compatibility is as 'gcc' for
    'vector bool/vector pixel' and as 'xl' for other types. Current default is
    'mixed'.

The name makes it clear that this has to do with source compatibility for Altivec vector types. The description is rather verbose - much more so than most options, but I think that is useful for an option such as this - that has a complex set of implications.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D103615