[PATCH] D94458: [PowerPC] Only use some extend mne if assembler is modern enough

Tue Jan 12 10:15:57 PST 2021

sfertile added a comment.

  clang -target powerpc64le-unknown-linux-gnu  -c -o a.o  modern-as.s
   error: instruction use requires an option to be enabled
              mfudscr 2

IIUC this patch is breaking compatibility with gas due to the AIX assembler not supporting the mnemonic. Similarly for my xxspltd patch, we wouldn't be able to encode the mnemonic without passing extra options on Linux. Thats why I took the approach I did in D94419 <https://reviews.llvm.org/D94419>. I am happy to change direction, but in my opinion we should only be disabling the mnemonics on AIX.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D94458