I committed this patch as 'obvious'. As I'm preparing the next round of future PowerPC patches, I noticed I had a typo in the usage for the predicate pcrel_external_address.
2019-08-01 Michael Meissner <meiss...@linux.ibm.com> * config/rs6000/predicates.md (pcrel_external_address): Update comment. Index: gcc/config/rs6000/predicates.md =================================================================== --- gcc/config/rs6000/predicates.md (revision 273981) +++ gcc/config/rs6000/predicates.md (working copy) @@ -1656,7 +1656,7 @@ (define_predicate "pcrel_address" ;; Return true if the operand is an external symbol whose address can be loaded ;; into a register using: -;; PLA reg,label@pcrel@got +;; PLD reg,label@pcrel@got ;; ;; The linker will either optimize this to either a PADDI if the label is ;; defined locally in another module or a PLD of the address if the label is -- Michael Meissner, IBM IBM, M/S 2506R, 550 King Street, Littleton, MA 01460-6245, USA email: meiss...@linux.ibm.com, phone: +1 (978) 899-4797